Skip to content

Fix --without-lsf and LSF in default search path #4227

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

Merged
merged 4 commits into from
Sep 22, 2017

Conversation

jjhursey
Copy link
Member

 * Spaces not tabs, and indent properly
 * No functional changes here

Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 39a83a2)
Signed-off-by: Joshua Hursey <[email protected]>
 * Will display a message acknowledging the configure setting
   instead of 'simple ok' which is misleading.

Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit fe97d3e)
Signed-off-by: Joshua Hursey <[email protected]>
…ach path

 * Reference Issue open-mpi#3546
 * If the user specified `--without-lsf` then do not check for it
   on the system, even if it is there. This can lead to the build
   failure identified in the issue above.

Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 24a8b5c)
Signed-off-by: Joshua Hursey <[email protected]>
@jjhursey jjhursey added the bug label Sep 18, 2017
@jjhursey jjhursey added this to the v2.1.3 milestone Sep 18, 2017
@hppritcha
Copy link
Member

@jjhursey could you add something to the NEWS for this PR?

@jjhursey
Copy link
Member Author

@hppritcha I just pushed a new commit that adds a note to NEWS.

@hppritcha
Copy link
Member

@jsquyres I think good to go.

@jsquyres jsquyres merged commit 46910c2 into open-mpi:v2.x Sep 22, 2017
@jjhursey jjhursey deleted the fix/v2.x/without-lsf branch September 22, 2017 18:43
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.

6 participants