-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 reliance on LegacyVersion #2822
Conversation
…on interpret_distro_name to be a tiny bit smarter about inferring the pivot separating name and version.
suggestion: When you do release this, do a major version bump. I expect that a lot of users who will complain that this is a breaking change (because their internal $corp systems depended on non-PEP 440 versions). The answer for them can then be (a) Python has been spewing out a deprecation warning about this for months, (b) it's a major version bump; deal with it, and (c) was there anything related to this change that would have made you reach out/react to it sooner? I'd strongly suggest asking the users what setuptools could have done to make the change less disruptive, rather than arguing whether the change is important -- the users will usually care mostly due to the immediate disruption it's caused them, and taking that moment to ask them "hey, how do you monitor for upcoming changes" is a useful exercise for both parties. :) |
Wait, there was never a release which spewed out deprecation warnings? Hmm... I think doing a major version series that does that is a worthwhile thing -- otherwise, this is basically a major change being made without any heads up to any of the downstream users; who sometimes can't control the exact version of setuptools they're using (eg: in pip's isolated build environments). Anyway, that's my two cents. I'mma step away and let y'all decide how you wanna manage changes around this project. :) |
Good idea. In #2840, I introduce such a warning. I'd expected that introducing that warning would have happened from the packaging package, but because it's vendored and pinned, that warning has been suppressed. |
Hiya @jaraco! Anything I can do to help move this forward? /cc @abravalheri as well. :) |
a85759c
to
93ce5a0
Compare
The deprecation has been in place for 15 months. Let's remove support. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a setuptools maintainer, but LGTM!
Summary of changes
Closes #2497.
Pull Request Checklist
changelog.d/
.(See documentation for details)