Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

{perf}[foss/2016a] Scalasca & Score-P update (REVIEW) #2892

Merged
merged 16 commits into from
May 15, 2016

Conversation

geimer
Copy link
Contributor

@geimer geimer commented Apr 14, 2016

Updated easyconfigs for the latest versions (released one hour ago) including deps.

Requires:

@hpcugentbot
Copy link
Contributor

Automatic reply from Jenkins: Can I test this?

@boegel
Copy link
Member

boegel commented Apr 14, 2016

Jenkins: ok to test

@geimer: try to limit this PR to 10 easyconfigs, if you need more, it's better to open a separate PR for a subgraph of the dependency tree, to keep things manageable

@boegel boegel added this to the v2.8.0 milestone Apr 14, 2016
@geimer
Copy link
Contributor Author

geimer commented Apr 14, 2016

@boegel: It'll be around 10, no worries.

@hpcugentbot
Copy link
Contributor

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7551/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
Member

boegel commented Apr 14, 2016

@geimer OK, give me a sign if you feel it's complete.

@hpcugentbot
Copy link
Contributor

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7554/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.

@geimer
Copy link
Contributor Author

geimer commented Apr 18, 2016

Although the build probably doesn't fail, easybuilders/easybuild-easyblocks#889 is needed to correctly enable all features.

@hpcugentbot
Copy link
Contributor

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7605/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

@boegel
Copy link
Member

boegel commented Apr 18, 2016

@geimer seems like this also depends on #2908 (which just got merged)

Please mention all required PRs in the PR description (which you can edit). Bonus points to 'cross out' merged PRs using ~~...~~.

@geimer
Copy link
Contributor Author

geimer commented Apr 18, 2016

@boegel: Yep, indeed. Forgot about it when pushing the latest changes...

@jgphpc jgphpc mentioned this pull request Apr 20, 2016
builds involving multiple toolchains (e.g., compiling Score-P for
Intel against Cube built with GGGcore)
@hpcugentbot
Copy link
Contributor

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7980/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.

@geimer
Copy link
Contributor Author

geimer commented May 11, 2016

Test report by @geimer
SUCCESS
Build succeeded for 8 out of 8 (8 easyconfigs in this PR)
swat - Linux debian 7.10, Intel(R) Xeon(R) CPU X5550 @ 2.67GHz, Python 2.7.3
See https://gist.github.com/3554899530b1511fe62becc044108726 for a full test report.

@geimer
Copy link
Contributor Author

geimer commented May 11, 2016

Tested on top of easybuilders/easybuild-easyblocks#889

@geimer geimer changed the title {perf}[foss/2016a] Scalasca & Score-P update (WIP) {perf}[foss/2016a] Scalasca & Score-P update (REVIEW) May 11, 2016
@@ -0,0 +1,52 @@
--- build-score/configure.orig 2016-05-11 09:21:03.716441529 +0200
Copy link
Member

Choose a reason for hiding this comment

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

@geimer: please include a comment above this line to explain what this patch does, and mention yourself as an author

prevent build-score/configure from picking up $CFLAGS etc., backported from Score-P 2.0.2
author: Markus Geimer (JSC)

@boegel
Copy link
Member

boegel commented May 14, 2016

@geimer: I ran into this when testing this PR:

../src/.libs/libunwind-x86_64.so: error: undefined reference to 'lzma_stream_footer_decode' 

It seems like XZ needs to be added as a dependency...

@boegel
Copy link
Member

boegel commented May 14, 2016

@geimer please see geimer#4, which handles the XZ issue (#3055 tackles the other existing libunwind easyconfigs)

@geimer
Copy link
Contributor Author

geimer commented May 15, 2016

Test report by @geimer
SUCCESS
Build succeeded for 9 out of 9 (8 easyconfigs in this PR)
swat - Linux debian 7.10, Intel(R) Xeon(R) CPU X5550 @ 2.67GHz, Python 2.7.3
See https://gist.github.com/5d62265636d375897761c2ca09d65448 for a full test report.

@boegel
Copy link
Member

boegel commented May 15, 2016

going in, thanks @geimer!

@boegel boegel merged commit ce8ae3f into easybuilders:develop May 15, 2016
@geimer geimer deleted the scalasca_update branch May 16, 2016 07:20
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.

None yet

3 participants