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
Don't import readline in isolated mode #72379
Comments
When running in isolated mode, readline should not be automatically imported (as it could conceivably be arbitrary code). |
Should also allow users to explicitly call site.enablerlcompleter() if they really want it. |
New changeset b0350f351752 by Steve Dower in branch '3.6': |
+1, but your patch is missing Misc/NEWS and a merge to 'default'. The code in Modules/main.c is getting harder to read, too. |
The NEWS change slipped into my following commit, and I just did the one merge. Since you're here, any ideas on how to test this? Adding a readline.py during the test suite will affect any tests running in parallel... |
The robot didn't mention your merge because the tracker id was not in the submit messages. I just saw it on python-cvs. How about a test in subprocess? You could drop a readline.py into Lib/tests/somedirectory and run sys.executable with cwd='Lib/tests/somedirectory'). |
Also, I think PEP-432 is the way to go about simplifying Py_Main, and I'm keen to see it happen (presumably for 3.7). But the time constraints make it hard, which is why Nick hasn't gotten it done yet either. |
New changeset 5761294bb877 by Steve Dower in branch '3.6': New changeset 6c7a8a012669 by Steve Dower in branch 'default': |
Turned out that we really only need to check for sys.__interactivehook__, which made it much easier to write the tests. |
Misc/NEWS
so that it is managed by towncrier #552Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: