-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Remove --global-option
and --build-option
flags
#12301
base: main
Are you sure you want to change the base?
Conversation
629b38a
to
2008ba1
Compare
16b0d66
to
2be580c
Compare
Perhaps we should postpone this, since setuptools does not seem to accept |
--global-options
and --build-option
--global-option
and --build-option
flags
I'm inclined to say that at some point, we need to bite the bullet and be explicit that this is a setuptools issue, not a pip one, and we won't maintain the legacy options indefinitely. But having said that, there's no particular reason we have to remove these now, so leaving them for a while longer is reasonable. The docs already note that the options only apply to the legacy |
2be580c
to
c227abc
Compare
Let's defer this for now -- I do agree that the reason for that is a setuptools issue and not a pip one, but I don't like the idea of leaving out users in a state where there isn't a way to achieve parity with an earlier release of pip due to us removing an option that serves a reasonable user-need but it's not served due to setuptools not being ready for it. I also don't like the idea of having folks point pitchforks at pip/setuptools maintainers for this, given that this doesn't really feel all that urgent and it's going to cause user churn when this gets removed anyway. Being able to say "here's what you need to change to" is definitely more palatable than "there is no equivalent to that thing". 😅
Maybe? It'd add friction to that removal though, and I think making this removal a slow burn vs a hard break is the choice we'd have to make here. My guess is that a slow burn around this (i.e. not coupling the two removals) would be a smoother rollout, but I'm also not (yet?) in possesion of a crystal ball that'd tell me the future. |
I think that the critical point here is what's being blocked on this. Removing the legacy So I'm fine with deferring, but if anyone feels that progress on something else is being delayed by leaving this (and the |
I think at this stage it only causes a slowly growing maintenance burden, but does not block pip evolution. So no urgency indeed. I postponed to 24.0 and we'll reevaluate then. |
Bump. This is currently scheduled for 24.2 which is due in ~a month. I'm not familiar with this functionality of pip so I don't have much to say other than we need to reevaluate whether we should remove these flags in 24.2 or continue to defer the removal (until setuptools can figure out a replacement for these flags?). /cc @sbidoul |
We can continue postponing. At some point we do want to remove setuptools special cases from pip, though, but there is no particular urgency for these options, knowing that we need to deprecate our uses of |
Coolio, I've deferred this to pip 25.0 (wait, it's going to be 2025 already 👀) . |
@ichard26 we'll need to |
Ah yes, of course. Thank you for reminding me of that :) Please see #12837. |
The removal of these options was scheduled for 23.3.
closes #11859