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

OpenMPI-2.*: remove --with-threads configure option. #4213

Merged

Conversation

bartoldeman
Copy link
Contributor

OpenMPI builds now warn they don't know --with-threads.
See OpenMPI commit at
open-mpi/ompi@7a55d49
sed -i 's/--with-threads=posix //' OpenMPI-2*.eb

OpenMPI builds now warn they don't know --with-threads.
See OpenMPI commit at
open-mpi/ompi@7a55d49
sed -i 's/--with-threads=posix //' OpenMPI-2*.eb
@boegel boegel added this to the 3.2.0 milestone Feb 28, 2017
@boegel
Copy link
Member

boegel commented Feb 28, 2017

lgtm

@boegel
Copy link
Member

boegel commented Feb 28, 2017

Test report by @boegel
SUCCESS
Build succeeded for 5 out of 5 (5 easyconfigs in this PR)
node2095.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/1ddfef157679f43e84a2038b6bc96b11 for a full test report.

@boegel boegel modified the milestones: 3.2.0, 3.1.1 Feb 28, 2017
@boegel
Copy link
Member

boegel commented Feb 28, 2017

Test report by @boegel
SUCCESS
Build succeeded for 10 out of 10 (5 easyconfigs in this PR)
node2482.golett.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/3def1b87f64807f058bca1e8ceff5e1b for a full test report.

@boegel
Copy link
Member

boegel commented Mar 1, 2017

Going in, thanks @bartoldeman!

@boegel boegel merged commit 12140e5 into easybuilders:develop Mar 1, 2017
@bartoldeman bartoldeman deleted the openmpi-remove-with-threads branch March 1, 2017 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants