gh-140287: Handle PYTHONSTARTUP script exceptions in the asyncio REPL
#140288
Conversation
|
Please merge #140298 first. |
bswck
requested review from
1st1,
asvetlov,
kumaraditya303 and
willingc
as code owners
| # Isolated mode implies -E, -P and -s, purifies sys.path and ignores PYTHON* | ||
| # variables. | ||
| if isolated: | ||
| cmd_line.append('-I') |
There was a problem hiding this comment.
To reuse this routine and spawn_asyncio_repl below, we need to have a knob to not pass -I since that ignores PYTHONSTARTUP completely.
The asyncio REPL currently doesn't comply to that, which is a separate issue I'll report having learnt this.
There was a problem hiding this comment.
Tracking the -I edge case in #140648.
| def test_asyncio_repl_reaches_python_startup_script(self): | ||
| with os_helper.temp_dir() as tmpdir: | ||
| script = os.path.join(tmpdir, "pythonstartup.py") | ||
| with open(script, "w") as f: | ||
| f.write("print('pythonstartup done!')" + os.linesep) | ||
| f.write("exit(0)" + os.linesep) | ||
|
|
||
| env = os.environ.copy() | ||
| env["PYTHON_HISTORY"] = os.path.join(tmpdir, ".asyncio_history") | ||
| env["PYTHONSTARTUP"] = script | ||
| subprocess.check_call( | ||
| [sys.executable, "-m", "asyncio"], | ||
| stdout=subprocess.PIPE, | ||
| stderr=subprocess.PIPE, | ||
| env=env, | ||
| timeout=SHORT_TIMEOUT, | ||
| ) |
There was a problem hiding this comment.
Covered by TestPythonStartup now, so not needed here.
|
I've gotten a little background on precedent changes to this region of the tests code. |
| for expected in ( | ||
| "Traceback (most recent call last):", | ||
| 'File "<stdin>", line 1, in <module>', | ||
| f'File "{startup_env['PYTHONSTARTUP']}", line ', |
There was a problem hiding this comment.
I need to understand why line number 2 fails specifically on Windows and 3 is expected.
1 participant
This is generally a work in progress; tests are needed.
The asyncio patch is simple and ready.
PYTHONSTARTUPexceptions #140287