Skip to content

Conversation

@sjeaugey
Copy link
Member

@sjeaugey sjeaugey commented May 22, 2018

Closes #4405

Copy link
Member

@jsquyres jsquyres left a comment

Choose a reason for hiding this comment

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

Go ahead and squash these two commits. Also might want to put "Closes #4405" in the description so that it closes the corresponding issue when it is merged. Thanks!

opal_show_help("help-mpi-common-cuda.txt", "dlopen failed", true,
errmsg);
if (opal_warn_on_missing_libcuda) {
opal_show_help("help-mpi-common-cuda.txt", "dlopen failed", true,
Copy link
Member

Choose a reason for hiding this comment

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

Super minor nit: should be a 4-space tab, not a 2-space tab.

@sjeaugey
Copy link
Member Author

@jsquyres spaces fixed and commits squashed.

Copy link
Member

@jsquyres jsquyres left a comment

Choose a reason for hiding this comment

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

Thanks!

Merge when CI completes.

@jsquyres
Copy link
Member

bot:ompi:retest

@jsquyres jsquyres merged commit 731fcc8 into open-mpi:master May 26, 2018
@jsquyres
Copy link
Member

@sjeaugey You might want to see if the RM's are amenable to you cherry picking this PR to the release branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants