➜
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2016-09-17 18:58 bysteve.dower, last changed2022-04-11 14:58 byadmin. This issue is nowclosed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 552 | closed | dstufft,2017-03-31 16:36 | |
| Messages (9) | |||
|---|---|---|---|
| msg276816 -(view) | Author: Steve Dower (steve.dower)* | Date: 2016-09-17 18:58 | |
When running in isolated mode, readline should not be automatically imported (as it could conceivably be arbitrary code). | |||
| msg276820 -(view) | Author: Steve Dower (steve.dower)* | Date: 2016-09-17 19:05 | |
Should also allow users to explicitly call site.enablerlcompleter() if they really want it. | |||
| msg276826 -(view) | Author: Roundup Robot (python-dev) | Date: 2016-09-17 19:57 | |
New changesetb0350f351752 by Steve Dower in branch '3.6':Issue#28192: Don't import readline in isolated modehttps://hg.python.org/cpython/rev/b0350f351752 | |||
| msg276829 -(view) | Author: Christian Heimes (christian.heimes)* | Date: 2016-09-17 20:01 | |
+1, but your patch is missingMisc/NEWS and a merge to 'default'. The code inModules/main.c is getting harder to read, too. | |||
| msg276831 -(view) | Author: Steve Dower (steve.dower)* | Date: 2016-09-17 20:02 | |
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... | |||
| msg276833 -(view) | Author: Christian Heimes (christian.heimes)* | Date: 2016-09-17 20:06 | |
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 intoLib/tests/somedirectory and run sys.executable with cwd='Lib/tests/somedirectory'). | |||
| msg276834 -(view) | Author: Steve Dower (steve.dower)* | Date: 2016-09-17 20:06 | |
Also, I thinkPEP 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. | |||
| msg276851 -(view) | Author: Roundup Robot (python-dev) | Date: 2016-09-17 21:35 | |
New changeset5761294bb877 by Steve Dower in branch '3.6':Issue#28192: Adds tests for hook in isolated modehttps://hg.python.org/cpython/rev/5761294bb877New changeset6c7a8a012669 by Steve Dower in branch 'default':Issue#28192: Adds tests for hook in isolated modehttps://hg.python.org/cpython/rev/6c7a8a012669 | |||
| msg276852 -(view) | Author: Steve Dower (steve.dower)* | Date: 2016-09-17 21:37 | |
Turned out that we really only need to check for sys.__interactivehook__, which made it much easier to write the tests. | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:37 | admin | set | github: 72379 |
| 2017-03-31 16:36:38 | dstufft | set | pull_requests: +pull_request1101 |
| 2016-09-24 19:22:02 | christian.heimes | link | issue12238 superseder |
| 2016-09-17 21:37:07 | steve.dower | set | status: open -> closed resolution: fixed messages: +msg276852 stage: needs patch -> resolved |
| 2016-09-17 21:35:57 | python-dev | set | messages: +msg276851 |
| 2016-09-17 20:06:09 | steve.dower | set | messages: +msg276834 |
| 2016-09-17 20:06:06 | christian.heimes | set | messages: +msg276833 |
| 2016-09-17 20:02:59 | steve.dower | set | messages: +msg276831 |
| 2016-09-17 20:01:43 | christian.heimes | set | type: behavior -> security components: + Interpreter Core |
| 2016-09-17 20:01:22 | christian.heimes | set | nosy: +christian.heimes messages: +msg276829 |
| 2016-09-17 19:57:29 | python-dev | set | nosy: +python-dev messages: +msg276826 |
| 2016-09-17 19:05:38 | steve.dower | set | messages: +msg276820 |
| 2016-09-17 18:58:34 | steve.dower | create | |