Skip to content

Conversation

@jsquyres
Copy link
Member

@jsquyres jsquyres commented May 7, 2016

Thanks to @PHHargrove for reporting in http://www.open-mpi.org/community/lists/devel/2016/05/18935.php.

Signed-off-by: Jeff Squyres [email protected]

@hjelmn Please review: is this sufficient? It looks like this component should build even if <sys/syscall.h> is not present, so I just added it to AC_CHECK_HEADERS.

Thanks to Paul Hargrove for reporting.

Signed-off-by: Jeff Squyres <[email protected]>
@jsquyres jsquyres added the bug label May 7, 2016
@jsquyres jsquyres added this to the v2.0.0 milestone May 7, 2016
@hjelmn
Copy link
Member

hjelmn commented May 7, 2016

This is failing env_mpi for some reason. Probably a SEGV.

@rhc54
Copy link
Contributor

rhc54 commented May 7, 2016

It's an related failure - you can ignore for now

@rhc54
Copy link
Contributor

rhc54 commented May 7, 2016

Sorry - "unrelated" failure

@hjelmn hjelmn merged commit b110f76 into open-mpi:master May 8, 2016
@hjelmn
Copy link
Member

hjelmn commented May 8, 2016

PR to 2.0.0

@PHHargrove
Copy link
Member

Two days ago hjelmn wrote:

PR to 2.0.0

I don't yet see such a PR.
-Paul

@jsquyres jsquyres deleted the pr/fix-memory-patcher-sys-syscall-h branch May 10, 2016 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants