Skip to content

Adjusted Cube download URL and tweaked build#1574

Merged
boegel merged 4 commits intoeasybuilders:developfrom
geimer:cube_update
Apr 29, 2015
Merged

Adjusted Cube download URL and tweaked build#1574
boegel merged 4 commits intoeasybuilders:developfrom
geimer:cube_update

Conversation

@geimer
Copy link
Copy Markdown
Contributor

@geimer geimer commented Apr 28, 2015

This change will make it possible to use --try-software-version and --filter-deps=Qt for Cube.
Requires easybuilders/easybuild-easyblocks#552

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 28, 2015

Automatic reply from Jenkins: Can I test this?

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 28, 2015

Jenkins: ok to test

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 28, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2909/
Test PASSed.

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 28, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2910/
Test PASSed.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 28, 2015

Test report by @boegel
FAILED
Build succeeded for 1 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/51b43e819f6757ba5a98 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 28, 2015

Contents of config.log for failed Cube configuration on top of easybuilders/easybuild-easyblocks#552 available @ https://gist.github.com/boegel/d01002dcbc16f33a7e10

@boegel boegel mentioned this pull request Apr 28, 2015
Merged
@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 29, 2015

config.log for build-backend in https://gist.github.com/boegel/b48c83b1f78a8fb40e3c

boegel
boegel reviewed Apr 29, 2015
View reviewed changes
Comment thread easybuild/easyconfigs/c/Cube/Cube-4.2-gompi-1.4.12-no-OFED.eb Outdated
Copy link
Copy Markdown
Member

@boegel boegel Apr 29, 2015

Choose a reason for hiding this comment

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

wouldn't it be better to just disable this then, since you told me only Tau uses this, and it is shipping it's own .jar for this?

either that, or include Java as a (build?) dependency here

@geimer
Copy link
Copy Markdown
Contributor Author

geimer commented Apr 29, 2015

OK, this build failure happens only if

  • Qt is also installed on the system as a package,
  • the system-provided Qt uses qmake-qt4 instead of qmake, and
  • one tries to install Cube 4.2 or 4.2.1 (due to a bug in the configure script).

The configure bug got fixed in Cube 4.2.2, thus I'm not sure now how to deal with this. Patching configure may be an option, though --try-software-version may lead to unexpected results. It is too bad that #1375 didn't make it in yet, so that we can tell people to use Cube 4.3...

@geimer
Try to fix build issue by patching configure
a67f2a8 
Provide updated easyconfig for Cube 4.3 (which should be used anyhow)
@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 29, 2015

Test report by @boegel
FAILED
Build succeeded for 0 out of 6 (3 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/bc7640edf613f0d15c87 for a full test report.

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 29, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2912/
Test PASSed.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 29, 2015

Test report by @boegel
FAILED
Build succeeded for 1 out of 3 (3 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/f0c4e7c6eff4f00a470d for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 29, 2015

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (3 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/4e250b0bb6d3dc044635 for a full test report.

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 29, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2913/
Test PASSed.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 29, 2015

Test report by @boegel
SUCCESS
Build succeeded for 8 out of 8 (8 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/fb9d42891040227312d0 for a full test report.

boegel added a commit to boegel/easybuild-easyconfigs that referenced this pull request Apr 29, 2015
@boegel
include PR easybuilders#1574 in release notes
5da3932
@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 29, 2015

Retested on top of easybuilders/easybuild-easyblocks#552, all green, so going in, thanks @geimer!

boegel added a commit that referenced this pull request Apr 29, 2015
@boegel
Merge pull request #1574 from geimer/cube_update
ccf5489 
Adjusted Cube download URL and tweaked build
@boegel boegel merged commit ccf5489 into easybuilders:develop Apr 29, 2015
@geimer geimer deleted the cube_update branch April 30, 2015 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@geimer @hpcugentbot @boegel