Skip to content
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.

Sign up for GitHub

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

Jump to bottom

v2.x: cleanup the os_dirpath logic #3345

Merged
merged 3 commits into from
Apr 21, 2017
Merged

v2.x: cleanup the os_dirpath logic #3345

merged 3 commits into from
Apr 21, 2017

Conversation

ggouaillardet
Copy link
Contributor

No description provided.

Ralph Castain and others added 3 commits April 13, 2017 16:31
@ggouaillardet
Cleanup the os_dirpath logic so it doesn't error out if the directory…
87c48f0 
… actually gets created (regardless of what mkdir returns), and pretty-prints the error if it does error out.

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
(cherry picked from commit b257c32)
(cherry picked from commit c722fb9)
@ggouaillardet
Quick fix: save the errno from the mkdir call as the call to stat wil…
6ce7c08 
…l likely overwrite it

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
(cherry picked from commit 6da4dbb)
(cherry picked from commit 867c841)
@ggouaillardet
opal/util: fix a race condition in opal_os_dirpath_create()
af52231 
always check the permissions of the created directory,
in case some one else created the very same directory but
with incompatible permissions

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>
(cherry picked from commit dffaad9)
(cherry picked from commit de539e0)
@ggouaillardet ggouaillardet added this to the v2.1.1 milestone Apr 13, 2017
@ggouaillardet ggouaillardet requested a review from rhc54 April 13, 2017 07:33
@ggouaillardet ggouaillardet mentioned this pull request Apr 13, 2017
Merged
@hppritcha hppritcha merged commit 113c322 into open-mpi:v2.x Apr 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rhc54 rhc54 rhc54 approved these changes

Assignees

@rhc54 rhc54

Labels
bug RM approved
Projects
None yet
Milestone
v2.1.1
Development

Successfully merging this pull request may close these issues.
3 participants
@ggouaillardet @rhc54 @hppritcha