Skip to content

Conversation

@hjelmn
Copy link
Member

@hjelmn hjelmn commented Apr 25, 2018

As @hjelmn and I discussed, this is a little hacky. However, it is the only solution that can be done solely from the OMPI side.

Signed-off-by: Ralph Castain rhc@open-mpi.org
(cherry picked from commit f424aa3)
Signed-off-by: Nathan Hjelm hjelmn@lanl.gov

As @hjelmn and I discussed, this is a little hacky. However, it is the only solution that can be done solely from the OMPI side.

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
(cherry picked from commit f424aa3)
Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>

(back-ported from commit open-mpi/ompi@edb8fe8)
Copy link
Contributor

@ggouaillardet ggouaillardet left a comment

Choose a reason for hiding this comment

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

looks good to me.

note I pushed an other commit into this PR related to PMIx v1.2 support

@bwbarrett
Copy link
Member

@ggouaillardet and @hjelmn when I merged #5081, this ended up with a conflict. Any chance one of you can take a look?

@rhc54
Copy link
Contributor

rhc54 commented Apr 26, 2018

@bwbarrett Fixed

@bwbarrett bwbarrett merged commit a905c7c into open-mpi:v3.1.x Apr 26, 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.

5 participants