Sorry, something went wrong.
|
@hoodmane you can always do manual backports using https://pypi.org/project/cherry_picker/ :) P.S. looks like you need to run make regen-cases to fix the CI. |
Sorry, something went wrong.
|
The backport can be done manually, just start the PR subject with [3.11] (the rest should be the same). And yes, please re-run regen-cases and commit Python/generated_cases.c.h. |
Sorry, something went wrong.
|
Okay updated generated_cases as requested. Thanks. |
Sorry, something went wrong.
|
GH-99514 is a backport of this pull request to the 3.11 branch. |
Sorry, something went wrong.
|
Thanks! |
Sorry, something went wrong.
⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️Hi! The buildbot wasm32-emscripten node (pthreads) 3.x has failed when building commit 86a49e0. What do you need to do:
You can take a look at the buildbot page here: https://buildbot.python.org/all/#builders/1050/builds/845 Failed tests:
Failed subtests:
Summary of the results of the build (if available): == Tests result: FAILURE == 338 tests OK. 10 slowest tests:
1 test failed: 94 tests skipped: Total duration: 21 min 14 sec Click to see traceback logsTraceback (most recent call last):
File "/opt/buildbot/bcannon-wasm/3.x.bcannon-wasm.emscripten-node-pthreads/build/Lib/test/test___all__.py", line 129, in test_all
self.check_all(modname)
File "/opt/buildbot/bcannon-wasm/3.x.bcannon-wasm.emscripten-node-pthreads/build/Lib/test/test___all__.py", line 43, in check_all
with warnings_helper.check_warnings(
File "/opt/buildbot/bcannon-wasm/3.x.bcannon-wasm.emscripten-node-pthreads/build/Lib/contextlib.py", line 144, in __exit__
next(self.gen)
File "/opt/buildbot/bcannon-wasm/3.x.bcannon-wasm.emscripten-node-pthreads/build/Lib/test/support/warnings_helper.py", line 185, in _filterwarnings
raise AssertionError("unhandled warning %s" % reraise[0])
AssertionError: unhandled warning {message : SyntaxWarning("invalid escape sequence '\\('"), category : 'SyntaxWarning', filename : '/opt/buildbot/bcannon-wasm/3.x.bcannon-wasm.emscripten-node-pthreads/build/Lib/test/test_enum.py', lineno : 1481, line : None}
|
Sorry, something went wrong.
|
@hoodmane Do you understand the buildbot failure? It seems related to this PR. |
Sorry, something went wrong.
|
It looks unrelated to me? It's hard for me to tell but it doesn't look like the -DPY_CALL_TRAMPOLINE compile flag was passed so the trampolines shouldn't be active and hence this change should look like a no-op to that build: checking LDSHARED... $(CC) -shared
checking BLDSHARED flags... $(CC) -shared
checking CCSHARED... -fPIC
checking LINKFORSHARED... -Xlinker -export-dynamic
checking CFLAGSFORSHARED...
I think if the trampolines were enabled we would see -DPY_CALL_TRAMPOLINE in one of these places. Is there a place I can see exactly the build bot did? |
Sorry, something went wrong.
|
Thanks @sobolevn! (I'm glad I didn't have to figure out what was going on...) |
Sorry, something went wrong.
|
GH-99514 is a backport of this pull request to the 3.11 branch. |
Sorry, something went wrong.
@tiran
Would appreciate a backport to 3.11.X. This code has been moved from ceval.c to bytecodes.c since the 3.11.X branch so it doesn't backport cleanly, but I have another patch available for 3.11. If someone tells me how I can make a separate backport PR.