{tools}[GCC/4.9.2] p7zip 9.38.1 (REVIEW)#1931
Conversation
|
Test report by @boegel |
|
Test report by @boegel |
|
Refer to this link for build results (access rights to CI server needed): This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
There was a problem hiding this comment.
yes, the 7z.so must be in the same directory as 7z, otherwise the latter doesn't work...
There was a problem hiding this comment.
ldd doesn't show that 7z requires 7z.so at all, so it goes hunting for 7z.so itself somehow (otherwise I would've just set $LD_LIBRARY_PATH)
so static linking can't help us here :)
note that 7z also doesn't work unless it's called full path (hence the alias below); otherwise, it complains that ./7z.so is not found... >_<
There was a problem hiding this comment.
what a fucking nightmare. Fine by me then.
There was a problem hiding this comment.
Fedora uses a simple script to achieve this:
#! /bin/sh
"/usr/libexec/p7zip/7z" "$@"|
Refer to this link for build results (access rights to CI server needed): This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
|
@wpoely86: please rereview? |
|
Test report by @boegel |
|
Refer to this link for build results (access rights to CI server needed): This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
|
Test report by @boegel |
There was a problem hiding this comment.
not sure about this one... what if the umask is set so that files are not supposed to be world-readable?
There was a problem hiding this comment.
so, actually, this should become chmod +x, i.e. enable execution bits for those allowed according to the current umask...
There was a problem hiding this comment.
or chmod +rx, but rwill already be enabled anyways
|
FWIW: while working on this , you might like to visit the MATLAB issues, too, since they are related:
Note that, having a GCC-less installation F. |
|
@fgeorgatos: I need this for another software package that comes as an I'll include a dummy easyconfig of |
|
Test report by @boegel |
|
Refer to this link for build results (access rights to CI server needed): This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
|
Test report by @boegel |
|
Test report by @boegel |
|
@wpoely86: please rererereview |
|
Refer to this link for build results (access rights to CI server needed): This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
|
looks fine |
|
Thanks for the review @wpoely86! |
{tools}[GCC/4.9.2] p7zip 9.38.1 (REVIEW)
No description provided.