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

v4.0.x: Allow user to overwrite OMPI_ENABLE_MPI1_COMPAT #6469

Conversation

bertwesarg
Copy link
Member

Follow-up to #6120.

As mentioned in [1], it may be desirable to nevertheless get the hidden
MPI 1 prototypes, for users who know what they are doing, i.e., the tools
guys. @ggouaillardet mentioned in [2], that -DOMPI_OMIT_MPI1_COMPAT_DECLS=0
should work, but it does not, as than we only get redefinition warnings.
See [3].

This topic does not relate to master, as we can remove the actual symbols
there, but here in v4.0.x land, the symbols are always there.

[1] #6120 (comment)
[2] #6120 (comment)
[3] #6120 (comment)

Signed-off-by: Bert Wesarg [email protected]

Follow-up to open-mpi#6120.

As mentioned in [1], it may be desirable to nevertheless get the hidden
MPI 1 prototypes, for users who know what they are doing, i.e., the tools
guys. @ggouaillardet mentioned in [2], that `-DOMPI_OMIT_MPI1_COMPAT_DECLS=0`
should work, but it does not, as than we only get redefinition warnings.
See [3].

This topic does not relate to master, as we can remove the actual symbols
there, but here in v4.0.x land, the symbols are always there.

[1] open-mpi#6120 (comment)
[2] open-mpi#6120 (comment)
[3] open-mpi#6120 (comment)

Signed-off-by: Bert Wesarg <[email protected]>
@bertwesarg
Copy link
Member Author

Please note, that I'm not allowed to set any reviewers and labels.

@ggouaillardet ggouaillardet requested a review from gpaulsen March 7, 2019 09:11
@ggouaillardet
Copy link
Contributor

@gpaulsen can you please review this ?

Since these symbols were removed from master this is a one-off commit for this release branch.

@gpaulsen
Copy link
Member

gpaulsen commented Mar 7, 2019

Yes, this looks good.
I don't understand why there isn't a configure switch for -DOMPI_OMIT_MPI1_COMPAT_DECLS=0.
I thought that was an internal variable (i.e. not set by users). But yes, this looks like a valid change if users are actively setting -DOMPI_OMIT_MPI1_COMPAT_DECLS=0

@bertwesarg
Copy link
Member Author

Yes, this looks good.
I don't understand why there isn't a configure switch for -DOMPI_OMIT_MPI1_COMPAT_DECLS=0.
I thought that was an internal variable (i.e. not set by users). But yes, this looks like a valid change if users are actively setting -DOMPI_OMIT_MPI1_COMPAT_DECLS=0

Because its to late for configure, tools work on an installed package.

@jsquyres
Copy link
Member

jsquyres commented Mar 7, 2019

The FAQ should be updated. And possibly the README...?

@bertwesarg bertwesarg deleted the v4.0.x-allow-user-to-overwrite-OMPI_ENABLE_MPI1_COMPAT branch March 7, 2019 20:52
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.

4 participants