-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ability to specify rust_extensions via setup.cfg #208
Comments
I agree that this would be great. I think that this needs support from |
FYI: a related upstream issue: pypa/setuptools#1272 |
Although we can not extend [[tool.setuptools-rust.rust-extensions]]
target = "example"
binding = "pyo3" Then we can implement a PEP 517 backend in setuptools-rust to read it and generate a corresponding [build-system]
requires = ["setuptools", "wheel", "setuptools-rust"]
build-backend = "setuptools_rust" |
At least for my use case I need the build backend to be setuptools itself, since I also have cffi extensions that need its setuptools extension. I don't we can do configuration in |
Right, but it should work after merging pyca/cryptography#7164 |
Good point! Though that's still a ways off since it needs patches to rust and pypy. |
Or we can add a [tool.setuptools-rust.extra-setup-args]
cffi_modules = ["src/_cffi_src/build_openssl.py:ffi"] |
I think it'd be also great if the use of the |
I saw the |
I think this is resolved now. Thank you! |
Currently one specifies
rust_extensions
insetup.py
, imperatively (e.g. https://github.com/pyca/cryptography/blob/main/setup.py#L43-L56). Nowadays, setuptools lets one specify many options declaratively insetup.cfg
(e.g. https://github.com/pyca/cryptography/blob/main/setup.cfg). It'd be great ifrust_extensions
could also be specified insetup.cfg
-- though truthfully I have no idea if that is possible!The text was updated successfully, but these errors were encountered: