Skip to content

ompi: disable functions removed from MPI-3.0 by default #5127

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

Merged
merged 3 commits into from
May 31, 2018

Conversation

hjelmn
Copy link
Member

@hjelmn hjelmn commented May 1, 2018

This commit adds a new configure option: --enable-mpi1-compat. Without
this option we will no longer provide APIs, typedefs, and defines that
were removed from the standard in MPI-3.0. This option will exist for
one major release (Open MPI v4.x.x) and then the option and associated
code will be removed in Open MPI v5.x.x. Open MPI has already
internally prepared for this change. Please prepare your codes
accordingly.

Signed-off-by: Nathan Hjelm hjelmn@lanl.gov

@ibm-ompi
Copy link

ibm-ompi commented May 1, 2018

The IBM CI (GNU Compiler) build failed! Please review the log, linked below.

Gist: https://gist.github.com/7173676ce0449dd15d4d2dd1de736707

@hjelmn hjelmn force-pushed the disable_removed branch from 34bc3f8 to 0551762 Compare May 2, 2018 02:23
@hjelmn
Copy link
Member Author

hjelmn commented May 2, 2018

bot:ompi:retest

@hjelmn
Copy link
Member Author

hjelmn commented May 2, 2018

Hah, failing on --enable-mpi-cxx. Won't bother fixing as that PR will probably go in before this one.

@hjelmn hjelmn added this to the v4.0.0 milestone May 2, 2018
hjelmn added 3 commits May 10, 2018 09:31
This commit adds a new configure option: --enable-mpi1-compat. Without
this option we will no longer provide APIs, typedefs, and defines that
were removed from the standard in MPI-3.0. This option will exist for
one major release (Open MPI v4.x.x) and then the option and associated
code will be removed in Open MPI v5.x.x. Open MPI has already
internally prepared for this change. Please prepare your codes
accordingly.

Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
@hjelmn hjelmn force-pushed the disable_removed branch from b43d10a to 0f919f8 Compare May 10, 2018 15:31
@hjelmn
Copy link
Member Author

hjelmn commented May 31, 2018

Looks like this is the way forward. Merging.

@hjelmn hjelmn merged commit b323655 into open-mpi:master May 31, 2018
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.

3 participants