Skip to content

Fix uninitialized variable. Set exit codes for failed launch so we get pretty error messages #3620

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 1 commit into from
May 31, 2017

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented May 31, 2017

Signed-off-by: Ralph Castain [email protected]

…t pretty error messages

Signed-off-by: Ralph Castain <[email protected]>
@rhc54
Copy link
Contributor Author

rhc54 commented May 31, 2017

@bwbarrett Sorry to be a pain, but the "default" CI is failing internally - something about resources like Botany Bay and others not being available?

@hjelmn
Copy link
Member

hjelmn commented May 31, 2017

botanybay was down part of the weekend due to electrical work. My house has some interesting wiring.... Will likely be down for about 6-8 hours the end of this week or next when I finish some new wiring.

@bwbarrett
Copy link
Member

botanybay being down should be fine; we now have two OS X builders (yay redundancy). I screwed up the PR builder yesterday while trying to fix how we build dependencies. I think I have it almost sorted out.

@bwbarrett
Copy link
Member

bot:ompi:retest

Unfortunately, the "default" was a bad configuration in Jenkins. There's no good way to remove it, so when the "Pull Request Build Checker" goes green, this is good to merge.

@rhc54
Copy link
Contributor Author

rhc54 commented May 31, 2017

roger that

@rhc54 rhc54 merged commit 4c00e2c into open-mpi:master May 31, 2017
@rhc54 rhc54 deleted the topic/grr branch May 31, 2017 20:08
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