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

v5.0.x: configure: Add --with-mpi-moduledir configure CLI option #12659

Merged

Conversation

jsquyres
Copy link
Member

@jsquyres jsquyres commented Jul 9, 2024

Parameterize the install location of Fortran MPI bindings modulefiles via the configure --with-mpi-moduledir CLI option (default to $libdir, per all prior versions of Open MPI).

Signed-off-by: Jeff Squyres [email protected]
(cherry picked from commit 7338594)

This is the v5.0.x PR corresponding to main PRs #12649 and #12667.

Fixes #12600

FYI @minrk @dalcinl @jeffhammond

@github-actions github-actions bot added this to the v5.0.4 milestone Jul 9, 2024
Parameterize the install location of Fortran MPI bindings modulefiles
via the configure --with-mpi-moduledir CLI option (default to $libdir,
per all prior versions of Open MPI).

Signed-off-by: Jeff Squyres <[email protected]>
(cherry picked from commit 7338594)
As a follow on to 7338594, add some additional information about
the --with-mpi-moduledir configure CLI option, especially as it
pertains to making relocatable Open MPI installations.

Also, remove an errant space that crept into the mpifort wrapper
compiler flags in 7338594.

Signed-off-by: Jeff Squyres <[email protected]>
(cherry picked from commit 1050509)
@jsquyres jsquyres force-pushed the pr/v5.0.x/move-fortran-modules-install branch from 821f6ab to 479c1c3 Compare July 11, 2024 14:43
@jsquyres
Copy link
Member Author

Rebased and added commit from main PR #12667.

@jsquyres jsquyres requested a review from bwbarrett July 11, 2024 14:44
@wenduwan wenduwan merged commit 4f1e29b into open-mpi:v5.0.x Jul 11, 2024
17 checks passed
@jsquyres jsquyres deleted the pr/v5.0.x/move-fortran-modules-install branch July 11, 2024 16:11
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.

None yet

4 participants