Skip to content

{tools}[GCC/4.9.2] p7zip 9.38.1 (REVIEW)#1931

Merged
boegel merged 5 commits intoeasybuilders:developfrom
boegel:p7zip_gcc492
Sep 1, 2015
Merged

{tools}[GCC/4.9.2] p7zip 9.38.1 (REVIEW)#1931
boegel merged 5 commits intoeasybuilders:developfrom
boegel:p7zip_gcc492

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Aug 31, 2015

No description provided.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 31, 2015

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/3c720f6f984060c913a8 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 31, 2015

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux centos linux 7.1.1503, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/74e2111b9109f7504aa9 for a full test report.

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4229/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4229/console for more details).

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.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so in the bin dir?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, the 7z.so must be in the same directory as 7z, otherwise the latter doesn't work...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

static linking is no option?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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... >_<

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what a fucking nightmare. Fine by me then.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fedora uses a simple script to achieve this:

#! /bin/sh
"/usr/libexec/p7zip/7z" "$@"

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4230/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4230/console for more details).

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.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 31, 2015

@wpoely86: please rereview?

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 31, 2015

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux centos linux 7.1.1503, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/b3cede6da539fbcca2ee for a full test report.

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4231/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4231/console for more details).

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.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 31, 2015

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/9e128dd9c320382bb6b2 for a full test report.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a+rx?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure about this one... what if the umask is set so that files are not supposed to be world-readable?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so, actually, this should become chmod +x, i.e. enable execution bits for those allowed according to the current umask...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or chmod +rx, but rwill already be enabled anyways

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay for +x

@fgeorgatos
Copy link
Copy Markdown
Contributor

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 p7 may be desirable, so that you may install fi. MATLAB, without requiring building GCC in advance. (IMHO, either dummy build or, tarball extract).

F.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 1, 2015

@fgeorgatos: I need this for another software package that comes as an .iso, so it is kind of relevant

I'll include a dummy easyconfig of p7zip in here as well.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 1, 2015

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/87808a5d3848a842bbea for a full test report.

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4232/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4232/console for more details).

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.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 1, 2015

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/07045a55592e284a9855 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 1, 2015

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
Linux centos linux 7.1.1503, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/d7b5da0a4ae5a2466d5e for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 1, 2015

@wpoely86: please rererereview

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4233/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4233/console for more details).

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
Copy link
Copy Markdown
Member

wpoely86 commented Sep 1, 2015

looks fine

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 1, 2015

Thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Sep 1, 2015
{tools}[GCC/4.9.2] p7zip 9.38.1 (REVIEW)
@boegel boegel merged commit ecd30c8 into easybuilders:develop Sep 1, 2015
@boegel boegel deleted the p7zip_gcc492 branch September 1, 2015 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants