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.

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

FileNotebookManager: Use shutil.move() instead of os.rename() #5548

Merged
merged 1 commit into from Apr 9, 2014

Conversation

pankajp
Copy link
Contributor

@pankajp pankajp commented Apr 7, 2014

os.rename() fails in case the new path is on a different
filesystem. For example if a file in /tmp is tried to save
to home, it causes following error:

[Errno 18] Invalid cross-device link using os.rename

`os.rename()` fails in case the new path is on a different
filesystem. For example if a file in `/tmp` is tried to save
to home, it causes following error:

[Errno 18] Invalid cross-device link using os.rename
@minrk minrk added this to the 2.1 milestone Apr 7, 2014
minrk added a commit that referenced this pull request Apr 9, 2014
FileNotebookManager: Use shutil.move() instead of os.rename()
@minrk minrk merged commit 13d3d22 into ipython:master Apr 9, 2014
@pankajp pankajp deleted the notebook-rename-cross-device-link branch April 10, 2014 02:38
minrk added a commit that referenced this pull request Apr 16, 2014
…os.rename()

`os.rename()` fails in case the new path is on a different
filesystem. For example if a file in `/tmp` is tried to save
to home, it causes following error:

[Errno 18] Invalid cross-device link using os.rename
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
…vice-link

FileNotebookManager: Use shutil.move() instead of os.rename()
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.

None yet

2 participants