[vim/vim] runtime(python): fix compatibility with Python 3.13 (PR #15088)
Yaakov Selkowitz
_PyObject_CallFunction_SizeT was dropped in Python 3.13:
You can view, comment on, or merge this pull request online at:
https://github.com/vim/vim/pull/15088
Commit Summary
- 18839a1 runtime(python): fix compatibility with Python 3.13
File Changes
(1 file)
- M src/if_python3.c (14)
Patch Links:
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
K.Takata
Does this affect when the limited API is used?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Yaakov Selkowitz
PyObject_CallFunction is listed as part of the Limited API according to https://docs.python.org/3/c-api/stable.html#contents-of-limited-api but perhaps I didn't understand your question.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
zdohnal
Hi @k-takata ,
what do you mean by "VIm compiled with limited API" - do you mean "dynamic" configure option value for --enable-python3interp? If you do, I would say the patch preserves using the old API for older Pythons and uses the new one for 3.13.
Please let us know how we could change the patch - VIm is currently not building in Fedora Rawhide, because new Python 3.13 came into the compose, so we would like to help with the fix.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
K.Takata
@zdohnal I meant the --with-python3-stable-abi option. (Limited API == Stable ABI)
Ideally, if Vim is compiled with --enable-python3interp=dynamic --with-python3-stable-abi=3.8, it should work with any version of Python between 3.8 and 3.13.
This is mostly useful for Windows users, so we can include this PR for now, and can check the compatibility later (if it is difficult to check).
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
zdohnal
@k-takata the current Fedora releases have Python 3.12 and Centos Stream 9 has 3.9 - I've successfully built the current Vim on all mentioned platforms with the options you mentioned. Tried ':py3 print('hello')' in c9s and it worked. In Fedora, public tests which run Vim project test suite passed with the patch, so it looks good.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Christian Brabandt
Thanks all.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Christian Brabandt
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.