Issue 18064: IDLE: add current directory to open_module
Issue18064
Created on 2013-05-25 21:36 by terry.reedy, last changed 2022-04-11 14:57 by admin.
| Messages (3) | |||
|---|---|---|---|
| msg190012 - (view) | Author: Terry J. Reedy (terry.reedy) * ![]() |
Date: 2013-05-25 21:36 | |
The point of File / Open Module is to look at the source for a module that has been, could be, or will be imported. The problem is that open_module is executed in the idle process while imports are executed in the user process, with the current directory prepended to sys.module. The problem was already noted in the source, line 675. # XXX Ought to insert current file's directory in front of path The augmentation of sys.path and subsequent restoration of the original could be done in a context manager that might be useful elsewhere. For 3.3,3.4, I think the change should be done after the patch in #18055, to not invalidate that patch. It would need to located in a slightly different place in 2.7, which will not be changed by that issue. |
|||
| msg190057 - (view) | Author: Terry J. Reedy (terry.reedy) * ![]() |
Date: 2013-05-26 03:48 | |
Code also suggests that this function should be elsewhere, such as in iobinding. This should be in the same place as regular open, wherever that is, since it is an alternative open. Document should be upgraded a bit: "Load module opens Python-coded modules on sys.path in an editor window. The default name is the currently selected text, if any." Tests should be added with this issue. This is the smallest dialog, and the structure of the revised code (#18055) gives a skeleton for a testcase: test_python_module (window open), test_builtin_module (Exception), test_bad_path (different Exception), and maybe even test_bad loader. We should look at tkinter tests for simulating name entry and click. |
|||
| msg296372 - (view) | Author: Terry J. Reedy (terry.reedy) * ![]() |
Date: 2017-06-19 20:38 | |
For IDLE development, I would love to have idlelib and idlelib.idle_test on the open line. Perhaps this should be user configurable: Favorite directories for opening files. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:57:46 | admin | set | github: 62264 |
| 2020-06-06 22:20:53 | terry.reedy | set | versions: + Python 3.10, - Python 3.6, Python 3.7 |
| 2017-06-19 20:38:43 | terry.reedy | set | assignee: terry.reedy components: + IDLE versions: + Python 3.6, Python 3.7, - Python 2.7, Python 3.3, Python 3.4 |
| 2017-06-19 20:38:22 | terry.reedy | set | messages: + msg296372 |
| 2015-09-18 16:31:15 | markroseman | set | nosy:
+ markroseman |
| 2013-05-26 03:48:37 | terry.reedy | set | messages: + msg190057 |
| 2013-05-25 21:36:38 | terry.reedy | set | dependencies: + Stop using imp in IDLE |
| 2013-05-25 21:36:20 | terry.reedy | create | |
