Skip to content

Use --oversubscribe not --oversubscribe with openmpi #265

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.

Sign up for GitHub

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

Jump to bottom
Closed
amckinstry opened this issue Dec 3, 2016 · 2 comments
Closed

Use --oversubscribe not --oversubscribe with openmpi #265

amckinstry opened this issue Dec 3, 2016 · 2 comments
Assignees
@zbeekman

Comments

@amckinstry
Copy link

Tests fail on building open-coarrays 1.7.4 on Debian unstable with openmpi 2.0.1 +.
When testing in a VM, CMakeLists.txt adds "--oversubscribed" where numcores is not enough.
The correct syntax on openmpi is "--oversubscribe"
@zbeekman
Copy link
Collaborator

zbeekman commented Dec 4, 2016

Thanks for pointing this out! I'll fix it asap

@zbeekman zbeekman mentioned this issue Dec 4, 2016
Closed
@zbeekman zbeekman self-assigned this Dec 4, 2016
@ghost ghost added needs-review and removed in-progress labels Dec 4, 2016
zbeekman added a commit that referenced this issue Dec 5, 2016
@zbeekman
Merge branch 'master' into bugfix-#265
5c54981
zbeekman added a commit that referenced this issue Dec 5, 2016
@zbeekman
Merge pull request #270 from sourceryinstitute/bugfix-#265
2523653 
Fixes #265 OpenMPI test launching bug

 Thanks to @amckinstry for reporting this bug
@ghost ghost removed the needs-review label Dec 5, 2016
@zbeekman
Copy link
Collaborator

zbeekman commented Dec 5, 2016

This is fixed on master now... There are some more bug fixes and enhancements, but we're aiming to get a release out ASAP.

@zbeekman zbeekman mentioned this issue Mar 6, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zbeekman zbeekman

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zbeekman @amckinstry