Open davidhewitt opened 1 year ago
(posted issue without content originally, sorry)
I'm wondering if we should change features around so that abi3
aka Py_LIMITED_API
is set by default. To use the unlimited API functionality we then can have an unlimited-api
(or unstable-python-api
or any other bikeshedded name) feature.
Advantages:
unlimited-api
feature would be additive, which fits normal Rust idiomsDisadvantage would be churn induced on users and tools like maturin
. We could make the unstable-python-api
a default feature for a few versions to make the migration easier.
@messense I'd be interested to hear what you think of this from a packaging / builds side. @encukou perhaps you'd be willing to share Python core dev opinion on whether we should be encouraging stable api compatibility as a default?
IMO it's not a big issue for maturin
, we can detect pyo3
version and change the abi3
detection logic to
unstable-python-api
/unlimited-api
features means non-abi3It might be an issue for setuptools-rust
? Since you need to build with --py-limited-api=cpXY
for abi3.
I'm not against changing the default, as long as any change in behavior is clearly visible with new compile errors.
Please don't add "unstable" to the feature name, the API should be perfectly stable within the Python minor version that the extension is compiled for.
One note: it'd be useful if pyo3 still supported specifying the abi3 minimum version if it differs from the current python.
For cryptography on Windows and macOS we build abi-py36 wheels using newer Pythons.
One note: it'd be useful if pyo3 still supported specifying the abi3 minimum version if it differs from the current python.
Definitely; I was thinking the existing abi3-py3x
features would all remain, the meaning of them being unchanged. The abi3
feature could exist as a no-op for a while for backwards compatibility too.
@encukou perhaps you'd be willing to share Python core dev opinion on whether we should be encouraging stable api compatibility as a default?
Well I'm a biased CPython developer, so you might want other opinions, but as far as I'm concerned, go for it :)
Don't use the term unstable API, that might mean something slightly different soon. Unlimited sounds good.
I'd also like to see abi3 be the default. For naming, what about a feature called no-abi3
and features py38
, py39
, etc. to select the minimum target version in either case?
what about a feature called
no-abi3
I feel that no-abi3
seems to imply that it's opt-out of some feature rather than opt-in to version specific feature, given that features in Rust is additive it's a bit weird.
and features
py38
,py39
, etc. to select the minimum target version in either case?
I like this!
Considering the apparent bike-sheddability of this, I deferring such a policy decision to Maturin wouldn't be a simpler first step that would help a lot to move the ecosystem into this direction. Especially for newcomers who have not yet found about the PyO3 features themselves.
NB #2901 has indicated that if we did this, users embedding Python would get no benefit (they would effectively be tied to a version-specific Python anyway).
Not a blocker, just something to document I think.
A reason to want this: libraries can accidentally rely on features from the unlimited API and then have users fail when running abi3 builds. If the default build mode was abi3
then libraries wouldn't run into this footgun.
https://github.com/davidhewitt/pythonize/issues/59#issuecomment-2032911500
I think adding this would cause less friction with #4008. I'm using apis that aren't available with abi3
, so being able to gate behind unlimited-api
would make sense.
@davidhewitt I'm open to doing a bunch of the work implementing this, since I'd like to be able to use it. Do you have any thoughts about when we should aim to land this? Is it something that can happen parallel to the gil-refs work or should it wait until after?
I'm also reasonably keen for this but given it creates churn for user's builds when upgrading I think either:
I think with the soundness adjustments in 0.22 it's probably a bit mean, but maybe in 0.23?
I'll make a start with a draft PR, but I'll not expect it to land for 0.22 then.
Just realised that at the moment we have a divergence between Py_LIMITED_API
and feature = "abi3"
cfg flags, because it's possible to set abi3=false
through the PyO3 config file and then the state of --feature=abi3
gets ignored.
https://github.com/PyO3/pyo3/pull/4185#discussion_r1604740811
Might influence how we want to redesign things.
Unlimited API also adds more performant alternatives to APIs, and it will be sad if users will lose them by default.
I do agree with this. One thought I keep having is that we should separate the ABI and API concerns. The ABI only matters when packaging, and could be set via either setuptools-rust
or maturin
if building an abi3 wheel.
The API, on the other hand, we could restrict while still using the performant ABI.
This way:
abi3
Yup, sticking to limited API allows you to build for multiple specific CPython versions of Python plus also one for stable ABI. That means the extension would work with previews of the in-development CPython as soon as they come out (for testing). And it should reduce pressure on maintainers to re-build/re-release within a short window of each year when there's a new CPython 3.N release, since a package wouldn't need 3.N builds for all its dependencies.