Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Conversation

@hppritcha
Copy link
Member

PR #896 broke os-x build. This commit fixes the problem.

@edgargabriel
@jsquyres

Signed-off-by: Howard Pritchard howardp@lanl.gov

@edgargabriel
Copy link
Member

looks good. 👍

@edgargabriel
Copy link
Member

actually, how can I revert the reviewed statement? I think you have a typo in there. It should probably be

#ifdef HAVE_SYS_PARAM_H
#include <sys/param.h>
#endif
#ifdef HAVE_SYS_MOUNT_H
#include <sys/mount.h>
#endif

You have twice sys/mount.h in the include statement.

@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1240/ for details.

@hppritcha
Copy link
Member Author

thanks, I'll fix.

PR open-mpi#896 broke os-x build. This commit fixes the problem.

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
@hppritcha hppritcha force-pushed the topic/osx_fixes_for_lustre_war branch from 6446dc8 to ad496b8 Compare January 21, 2016 18:19
@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1241/ for details.

@hppritcha
Copy link
Member Author

@jsquyres this is ready to go

jsquyres added a commit that referenced this pull request Jan 21, 2016
@jsquyres jsquyres merged commit 8505538 into open-mpi:v2.x Jan 21, 2016
yburette pushed a commit to yburette/ompi-release that referenced this pull request Feb 24, 2016
Use a modified variant of open-mpi#907.  Thanks to plesn
for noticing this.

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
(cherry picked from commit open-mpi/ompi@a31cc21)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants