Skip to content

Conversation

@ggouaillardet
Copy link
Contributor

this fixes a regression introduced in 045d0c5

Fixes #2879

Signed-off-by: Gilles Gouaillardet gilles@rist.or.jp

(back-ported from commit 9bcadbd)

this fixes a regression introduced in open-mpi/ompi@045d0c5

Fixes open-mpi#2879

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>

(back-ported from commit open-mpi/ompi@9bcadbd)
@ggouaillardet
Copy link
Contributor Author

@hppritcha can you please double check the LANL-disable-dlopen failure ?
i was unable to reproduce the issue on openSUSE Leap 42.2

at the beginning of the ./autogen.pl output, i noted

2. Running template-generating scripts

=== ompi/include/mpif-values.pl
creating Fortran header files (with common constants)...
./ompi/include/mpif-handles.h unchanged; not written
./ompi/include/mpif-constants.h unchanged; not written
./ompi/include/mpif-io-handles.h unchanged; not written
./ompi/include/mpif-io-constants.h unchanged; not written
./ompi/mpi/fortran/use-mpi-f08/constants.h unchanged; not written

and i also cannot make any sense of this error ...
could it be caused by a dirty working directory ?

@jjhursey jjhursey self-requested a review January 30, 2017 16:50
Copy link
Member

@jjhursey jjhursey left a comment

Choose a reason for hiding this comment

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

👍 This looks good. Thanks for the fix!

@jsquyres
Copy link
Member

Per #2881, this is good.

@jsquyres jsquyres merged commit 353870a into open-mpi:v2.x Jan 30, 2017
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.

4 participants