-
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
Unable to control --debug
/ --release
via PEP517 backend
#327
Comments
I think you're correct. Unless things have changed, I don't think PEP517 gives a mechanism to pass arguments to the build backend in general. This was solved in I'd accept a PR to achieve a similar control via env var here, though the design might be a bit different. |
Actually, PEP517 provides a That said, I'm not really sure because the code is very complex and unreadable to me, and it's undocumented. |
Hmm, reading around a little, I think we can try to support something like
Looks like all PEP517 arguments to setuptools builds must go under the |
Thinking about it, I'd prefer environment variable approach — if only because it means we can set it globally and not have to worry whether the package in question uses setuptools-rust or not. |
Since 1.8.0 we now have Note that for |
Thanks! FWICS, maturin uses |
Yes I think |
Unless I'm mistaken, there is no way to pass
--debug
/--release
argument tobuild_rust
when building via the PEP517 backend.The text was updated successfully, but these errors were encountered: