Skip to content

Use modextrapaths instead of modextravars in OpenMM easyconfig#4903

Merged
boegel merged 2 commits intoeasybuilders:developfrom
pescobar:opemm-fix
Aug 10, 2017
Merged

Use modextrapaths instead of modextravars in OpenMM easyconfig#4903
boegel merged 2 commits intoeasybuilders:developfrom
pescobar:opemm-fix

Conversation

@pescobar
Copy link
Copy Markdown
Member

@pescobar pescobar commented Jul 17, 2017

No description provided.

@verdurin
Copy link
Copy Markdown
Member

verdurin commented Jul 18, 2017

@pescobar tried to run a test build of this, and the building of the dependency OpenBLAS/0.2.6-gompi-1.4.10-LAPACK-3.4.2 failed...

@verdurin verdurin added this to the 3.4.0 milestone Jul 18, 2017
@verdurin
Copy link
Copy Markdown
Member

verdurin commented Jul 18, 2017

Test report by @verdurin
FAILED
Build succeeded for 4 out of 18 (1 easyconfigs in this PR)
ca014.camp.thecrick.org - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2640 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/8339437fea3e690b489212e18256462c for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 10, 2017

@verdurin Yes, that's probably because that version of OpenBLAS is too old for the processor architecture of your system... We should deprecate/archive goolf/1.4.10 soon.

This change makes perfect sense though, and it doesn't affect the build itself, only the generated module, so testing with --module-only --force is actually sufficient here.

(full) test report from our Sandy Bridge-based cluster coming up...

@boegel boegel changed the title OpenMM. fix typo. Use modextrapaths instead of modextravars Use modextrapaths instead of modextravars in OpenMM easyconfig Aug 10, 2017
@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 10, 2017

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 10, 2017

Going in, thanks @pescobar!

@boegel boegel merged commit 57533d4 into easybuilders:develop Aug 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.4.0

Development

Successfully merging this pull request may close these issues.

3 participants

@pescobar @verdurin @boegel