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

update SCOTCH 6.0.5 easyconfigs in-place to 6.0.6 to fix wrong download URL #7159

Merged
merged 1 commit into from
Dec 7, 2018

Conversation

boegel
Copy link
Member

@boegel boegel commented Nov 14, 2018

fix for problem reported by @smoors in #7154

see also enhanced SCOTCH easyblock in easybuilders/easybuild-easyblocks#1580

WIP since needs thorough testing (which is under way); test reports welcome!

@boegel boegel added this to the 3.8.0 milestone Nov 14, 2018
@smoors
Copy link
Contributor

smoors commented Nov 14, 2018

Test report by @smoors
FAILED
Build succeeded for 41 out of 45 (11 easyconfigs in this PR)
nic420 - Linux centos linux 7.5.1804, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/f305a069642fa1e9312d53af6542dec6 for a full test report.

@boegel
Copy link
Member Author

boegel commented Nov 14, 2018

@smoors:

  • Trilinos failed because the required source tarball could not be downloaded because of https issues, you may need to download it manually?

  • similar issue for SUNDIALS

  • patchelf failed with bootstrap.sh: line 2: autoreconf: command not found, that looks like a missing build dep on Autotools, are you up for opening a PR to fix that?

@boegel
Copy link
Member Author

boegel commented Nov 15, 2018

Test report by @boegel
SUCCESS
Build succeeded for 11 out of 11 (11 easyconfigs in this PR)
node2726.swalot.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/da2e2aea77521fd8c1173d03eb70abee for a full test report.

@boegel
Copy link
Member Author

boegel commented Nov 15, 2018

Test report by @boegel
SUCCESS
Build succeeded for 12 out of 12 (11 easyconfigs in this PR)
node2079.delcatty.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/eaf8d94268a1f0f16a88e294587da2bb for a full test report.

@verdurin
Copy link
Member

@smoors ping on the fixes for this one?

@smoors
Copy link
Contributor

smoors commented Nov 26, 2018

fix for patchelf: #7175
downloading SUNDIALS and Trilinos appears work again.

@boegel
Copy link
Member Author

boegel commented Nov 29, 2018

@smoors Any idea what the problem is with the failing builds?

@boegel boegel changed the title update SCOTCH 6.0.5 easyconfigs in-place to 6.0.6 to fix wrong download URL (WIP) update SCOTCH 6.0.5 easyconfigs in-place to 6.0.6 to fix wrong download URL Nov 29, 2018
@smoors
Copy link
Contributor

smoors commented Nov 30, 2018

Test report by @smoors
SUCCESS
Build succeeded for 11 out of 11 (11 easyconfigs in this PR)
nic31 - Linux centos linux 7.5.1804, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/c09974956b508db151bbf1b4f129c0f9 for a full test report.

@migueldiascosta
Copy link
Member

with current develop here, --from-pr for this PR fails with

ERROR: Couldn't find path to patched file /dev/shm/tmp/eb-FlHmx/files_pr7159/s/SCOTCH/SCOTCH-6.0.6-foss-2018a.eb

It works in 3.7.1 though...

@boegel
Copy link
Member Author

boegel commented Dec 5, 2018

@migueldiascosta Follow-up on that in easybuilders/easybuild-framework#2679

@boegel
Copy link
Member Author

boegel commented Dec 5, 2018

@migueldiascosta Problem should be fixed with easybuilders/easybuild-framework#2680

Copy link
Member

@migueldiascosta migueldiascosta left a comment

Choose a reason for hiding this comment

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

lgtm

@migueldiascosta
Copy link
Member

Going in, thanks @boegel!

@migueldiascosta migueldiascosta merged commit 48b2e65 into easybuilders:develop Dec 7, 2018
@boegel boegel deleted the SCOTCH_version_fix branch December 7, 2018 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants