Skip to content

{toolchain} [iomkl/2015.01] (OK)#1325

Merged
boegel merged 3 commits intoeasybuilders:developfrom
pescobar:iomkl-2015.01
Jan 22, 2015
Merged

{toolchain} [iomkl/2015.01] (OK)#1325
boegel merged 3 commits intoeasybuilders:developfrom
pescobar:iomkl-2015.01

Conversation

@pescobar
Copy link
Copy Markdown
Member

@pescobar pescobar commented Jan 22, 2015

No description provided.

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jan 22, 2015

Test PASSed.

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jan 22, 2015

Test PASSed.

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Jan 22, 2015

Test report by @pescobar
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
Linux centos 6.5, Intel(R) Xeon(R) CPU E5-2620 0 @ 2.00GHz, Python 2.6.6
See https://gist.github.com/0b824548d62f1de7bdd7 for a full test report.

boegel
boegel reviewed Jan 22, 2015
View reviewed changes
Comment thread easybuild/easyconfigs/i/iomkl/iomkl-2015.01.eb Outdated
Copy link
Copy Markdown
Member

@boegel boegel Jan 22, 2015

Choose a reason for hiding this comment

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

no need for defining ompi, since OpenMPI is only used in one place?

dependencies = [
    ...
    ('OpenMPI', '1.8.4', '', comp),
   ...
]

Copy link
Copy Markdown
Member

@boegel boegel Jan 22, 2015

Choose a reason for hiding this comment

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

actually, same goes for comp, so drop that too and use (compver is still useful):

dependencies = [
    ...
    ('OpenMPI', '1.8.4', '', ('iccifort', compver)),
   ...
]

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 22, 2015

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

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Jan 22, 2015

Test PASSed.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 22, 2015

Looking great @pescobar, going in. Thanks!

boegel added a commit that referenced this pull request Jan 22, 2015
@boegel
Merge pull request #1325 from pescobar/iomkl-2015.01
b8c2e1b 
{toolchain} [iomkl/2015.01] (REVIEW)
@boegel boegel merged commit b8c2e1b into easybuilders:develop Jan 22, 2015
@boegel boegel changed the title {toolchain} [iomkl/2015.01] (REVIEW) {toolchain} [iomkl/2015.01] (OK) Jan 22, 2015
@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Jan 22, 2015

Test report by @pescobar
SUCCESS
Build succeeded for 0 out of 0 (6 easyconfigs in this PR)
Linux centos 6.5, Intel(R) Xeon(R) CPU E5-2620 0 @ 2.00GHz, Python 2.6.6
See https://gist.github.com/9366de3452a0fd31ab20 for a full test report.

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Jan 22, 2015

Test report by @pescobar
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
Linux centos 6.5, Intel(R) Xeon(R) CPU E5-2620 0 @ 2.00GHz, Python 2.6.6
See https://gist.github.com/5b41889dc3cc35c86afd for a full test report.

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

@pescobar @hpcugentbot @boegel