-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Introduce variables for MPI compiler wrappers and document their usage #893
Conversation
f3c5316
to
f417b1c
Compare
@adamjstewart @mplegendre ping. |
Don't forget mvapich2 |
done. |
@davydden: looks good to me -- can you move the |
@tgamblin done. I also removed
did not reveal its usage in any package. |
in the spirit of #657 .
tested with
spack install p4est ^mpich
andspack install p4est ^openmpi
. Both compile fine.p4est ^mpich
have a few failed checks, but it is a separate issue not related to this PR.@eschnett @alalazo @citibeth @lee218llnl @mathstuf @tgamblin ping.
fixes #572