-
-
Notifications
You must be signed in to change notification settings - Fork 32k
gh-116869: Enable test_cext and test_cppext on Free Threading build #116973
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
Conversation
!buildbot nogil |
🤖 New build scheduled with the buildbot fleet by @colesbury for commit 98ff887 🤖 The command will test the builders whose names match following regular expression: The builders matched are:
|
Thanks, I planned to do it, but waited for the first basic checks to make sure that I didn't mess up anything :-) |
Lib/test/test_cext/__init__.py
Outdated
@unittest.skipIf( | ||
'-fsanitize' in (sysconfig.get_config_var('PY_CFLAGS') or ''), | ||
'test does not work with analyzing builds') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can probably use support.skip_if_sanitizer(...)
here
…uild Remove the "if Py_GIL_DISABLED" skip and move all "skip" decorators to the class.
98ff887
to
1a3a98b
Compare
Oh, I had to rebase this PR to retrieve the WASI build fix. |
…uild (python#116973) Remove the "if Py_GIL_DISABLED" skip and move all "skip" decorators to the class. Use support.check_sanitizer()
…uild (python#116973) Remove the "if Py_GIL_DISABLED" skip and move all "skip" decorators to the class. Use support.check_sanitizer()
…uild (python#116973) Remove the "if Py_GIL_DISABLED" skip and move all "skip" decorators to the class. Use support.check_sanitizer()
Remove the "if Py_GIL_DISABLED" skip and move all "skip" decorators to the class.