Skip to content

Conversation

@ggouaillardet
Copy link
Contributor

Fixes #3065

@ggouaillardet ggouaillardet force-pushed the topic/v2.0.x/mpi_pack_unpack branch from f597660 to 59b20e1 Compare March 1, 2017 01:41
@ggouaillardet
Copy link
Contributor Author

this is an alternative to #3061

@ggouaillardet ggouaillardet changed the title fix and revamp MPI_Pack and friends v2.0.x: fix and revamp MPI_Pack and friends Mar 1, 2017
@ggouaillardet ggouaillardet added this to the v2.0.3 milestone Mar 1, 2017
@ggouaillardet ggouaillardet requested a review from bosilca March 1, 2017 02:37
Copy link
Member

@bosilca bosilca left a comment

Choose a reason for hiding this comment

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

Same comment as #3070

@jsquyres
Copy link
Member

jsquyres commented Mar 1, 2017

@hppritcha This is good to go.

ggouaillardet and others added 5 commits March 2, 2017 09:25
…type

so it can be directly used by test/datatype/external32

(back-ported from commit open-mpi/ompi@5932287)

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>
this regression was previously introduced in open-mpi/ompi@221e6e2

(back-ported from commit open-mpi/ompi@f2e33c7)

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>
this fixes a regression from open-mpi/ompi@f2e33c7

(cherry picked from commit open-mpi/ompi@7b803ac)
Thanks Alex and the folks at Mellanox for the help.

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

(back-ported from commit open-mpi/ompi@880f2d5)
@jsquyres
Copy link
Member

jsquyres commented Mar 2, 2017

Travis is having an outage tonight and builds are currently paused. I'm merging so that this fix gets in for tonight's MTT runs.

@jsquyres jsquyres merged commit ea92775 into open-mpi:v2.0.x Mar 2, 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.

3 participants