Skip to content

[3.15] gh-140550: Enable limited C API tests on Free Threading in test_cext (GH-157493) - #157560

Merged
serhiy-storchaka merged 1 commit into
python:3.15from
miss-islington:backport-fb123c8-3.15
Oct 10, 2026
Merged

serhiy-storchaka merged 1 commit into
python:3.15from
miss-islington:backport-fb123c8-3.15

Conversation

@miss-islington

@miss-islington miss-islington commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

Enable limited C API tests on Free Threading in test_cext and
test_cppext: test Py_TARGET_ABI3T macro.

  • Convert test_cppext to PySlot API using PySlot_PTR_STATIC().
  • Define Py_MOD_GIL_NOT_USED in test_cext and test_cppext.
  • Disable C++ test_virtual_object() if Py_TARGET_ABI3T is defined.
    (cherry picked from commit fb123c8)

Co-authored-by: Victor Stinner vstinner@python.org

…_cext (pythonGH-157493)

Enable limited C API tests on Free Threading in test_cext and
test_cppext: test Py_TARGET_ABI3T macro.

* Convert test_cppext to PySlot API using PySlot_PTR_STATIC().
* Define Py_MOD_GIL_NOT_USED in test_cext and test_cppext.
* Disable C++ test_virtual_object() if Py_TARGET_ABI3T is defined.
(cherry picked from commit fb123c8)

Co-authored-by: Victor Stinner <vstinner@python.org>

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. This change can wait for Python 3.15.1 bugfix release.

@vstinner

Copy link
Copy Markdown
Member

Note to myself: it would be safer to run buildbot checks once the 3.15 branch will be unlocked.

@serhiy-storchaka
serhiy-storchaka merged commit 96f7177 into python:3.15 Oct 10, 2026
60 checks passed
@miss-islington
miss-islington deleted the backport-fb123c8-3.15 branch October 10, 2026 10:20
@vstinner

Copy link
Copy Markdown
Member

Thanks for the merge @serhiy-storchaka. I will keep an eye on https://buildbot.python.org/#/release_status since test_cext is skipped on GitHub Actions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants