Skip to content

Fix compilation on free-threaded Python 3.14t#107

Merged
jensens merged 3 commits intozopefoundation:masterfrom
bluedynamics:fix-freethreaded
Mar 17, 2026
Merged

Fix compilation on free-threaded Python 3.14t#107
jensens merged 3 commits intozopefoundation:masterfrom
bluedynamics:fix-freethreaded

Conversation

@jensens
Copy link
Copy Markdown
Member

@jensens jensens commented Feb 16, 2026

Summary

  • Replace `ob_refcnt` direct access with `Py_REFCNT()` macro in `FREE_ARG_TUP` macro (line 720)
  • Replace `obj->ob_type->tp_name` with `Py_TYPE(obj)->tp_name` (line 1528)
  • Add 3.14t to CI matrix and tox envlist

Tested locally: compiles and all tests pass on both Python 3.14 and 3.14t.

Refs #106

- Use Py_REFCNT() instead of direct ob_refcnt struct access
- Use Py_TYPE() instead of direct ob_type struct access
- Add 3.14t to CI and tox envlist

Refs zopefoundation#106
Copy link
Copy Markdown
Member

@davisagli davisagli left a comment

Choose a reason for hiding this comment

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

C changes LGTM. Needs to be updated with zopefoundation/meta#389

@jensens jensens marked this pull request as ready for review March 9, 2026 23:14
@jensens jensens enabled auto-merge March 9, 2026 23:18
@jensens
Copy link
Copy Markdown
Member Author

jensens commented Mar 9, 2026

C changes LGTM. Needs to be updated with zopefoundation/meta#389

updated

@jensens
Copy link
Copy Markdown
Member Author

jensens commented Mar 11, 2026

updated with zope meta again

@jensens jensens requested a review from davisagli March 11, 2026 14:23
@jensens jensens merged commit 361aecb into zopefoundation:master Mar 17, 2026
51 checks passed
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.

2 participants