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
ensure scp destination directories exist (with mkdir -p) #4977
Conversation
I think there is os.path.makedirs() for the local case. |
in SSH launchers. As described in ipython#3411 closes ipython#3411
Right, uses |
@@ -623,6 +628,9 @@ def _fetch_file(self, remote, local): | |||
time.sleep(1) | |||
elif check == u'yes': | |||
break | |||
local_dir = os.path.dirname(local) | |||
if not os.path.exists(local_dir): | |||
os.makedirs(local_dir) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only now I look at the docs, I see directories are created in mode 777 (world writable) by default. I'm not sure what files might be fetched, but I guess we might want something like 775.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After the dir permissions are changed will this one be ready for merging? |
permissions changed |
Looks good, merging. |
ensure scp destination directories exist (with mkdir -p)
ensure scp destination directories exist (with mkdir -p)
in SSH launchers.
As described in #3411
closes #3411