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

%edit files mistakenly import modules in /tmp #4731

Closed

mgummelt opened this issue Dec 22, 2013 · 1 comment · Fixed by #4848

Milestone

mgummelt commented Dec 22, 2013

files created by %edit are stored in /tmp, so they mistakenly import modules from this directory if there is a name collision.

Member

minrk commented Dec 22, 2013

I guess we should make a temporary directory as well.

Carreau added a commit to Carreau/ipython that referenced this issue


          avoid import of nearby temporary with %edit

73df570

use mk**s**temp now,
and create files in subfolders.

should close ipythongh-4731

Note that some part of the logic could now use
http://docs.python.org/2/library/tempfile.html#tempfile.NamedTemporaryFile
That will be deleted on close and avoid IPython to track it.

Carreau mentioned this issue

avoid import of nearby temporary with %edit #4848

Merged

Carreau added a commit to Carreau/ipython that referenced this issue


          avoid import of nearby temporary with %edit

d1296e5

use mk**s**temp now,
and create files in subfolders.

should close ipythongh-4731

Note that some part of the logic could now use
http://docs.python.org/2/library/tempfile.html#tempfile.NamedTemporaryFile
That will be deleted on close and avoid IPython to track it.

takluyver closed this as completed in #4848

mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue


          avoid import of nearby temporary with %edit

09b7763

use mk**s**temp now,
and create files in subfolders.

should close ipythongh-4731

Note that some part of the logic could now use
http://docs.python.org/2/library/tempfile.html#tempfile.NamedTemporaryFile
That will be deleted on close and avoid IPython to track it.

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