Skip to content
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

install options have no effect? #185

Open
haampie opened this issue Nov 14, 2022 · 1 comment
Open

install options have no effect? #185

haampie opened this issue Nov 14, 2022 · 1 comment

Comments

@haampie
Copy link

haampie commented Nov 14, 2022

I have the impression that none of this:

# PyPy doesn't support the C backend.
if platform.python_implementation() == "PyPy":
C_BACKEND = False
if "--legacy" in sys.argv:
SUPPORT_LEGACY = True
sys.argv.remove("--legacy")
if "--system-zstd" in sys.argv:
SYSTEM_ZSTD = True
sys.argv.remove("--system-zstd")
if "--warnings-as-errors" in sys.argv:
WARNINGS_AS_ERRORS = True
sys.argv.remove("--warning-as-errors")
if "--no-c-backend" in sys.argv:
C_BACKEND = False
sys.argv.remove("--no-c-backend")
if "--no-cffi-backend" in sys.argv:
CFFI_BACKEND = False
sys.argv.remove("--no-cffi-backend")
if "--rust-backend" in sys.argv:
RUST_BACKEND = True
sys.argv.remove("--rust-backend")

actually does anything.

I can pass --install-option=--no-c-backend but it is ignored.

@DjDeepDive
Copy link

Furthermore, checking for sys.argv directly is brittle, as PIP is deprecating the ability to pass arguments this way at all. This should all be changed to --config-settings KEY=VALUE format if possible:

DEPRECATION: --build-option and --global-option are deprecated. pip 23.3 will enforce this behaviour change. A possible replacement is to use --config-settings. Discussion can be found at https://github.com/pypa/pip/issues/11859

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants