hynek / argon2-cffi-bindings

Low-level Python CFFI Bindings for Argon2
MIT License
8 stars 9 forks source link

Add 3.13 to CI #57

Closed hynek closed 3 months ago

hynek commented 7 months ago

We'll need a CFFI release first, though.

ref https://github.com/hynek/argon2-cffi/pull/175

hynek commented 3 months ago

I do not plan on merging as it is right now until we got a public 1.17.

There should be no changes necessary once they do they release.

minrk commented 3 months ago

FWIW, free-threaded CPython 3.13 doesn't support the limited API, so to you'll need to set py_limited_api=False when sysconfig.get_config_var("Py_GIL_DISABLED"), both in setup.py and ffi.set_source, I believe, in order to get this package to install for cp313t builds.

CFFI doesn't officially support free-threaded Python yet, so perhaps not much reason to pursue it now. That said, I've successfully installed argon2-cffi with

hynek commented 3 months ago

In principle, CFFI always release the GIL , so I hope that it’s gonna be just a matter of compilation flags. 😬

tacaswell commented 2 months ago

I also needed https://github.com/python-cffi/cffi/pull/112