-
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
Update legacy editables support for compatibility with setuptools >= 69 #12477
Conversation
f00a2be
to
630ba15
Compare
Obvious question, but is there an easy way to run the Pip test suite using setuptools<69? Some users pin their build dependencies either for their own project via |
630ba15
to
98dc4b2
Compare
You can test that by constraining setuptools in |
Thanks, I tested this branch locally using I did have one flaky test |
Thanks for confirming @notatallshaw. I have not experienced the flakyness myself, though. |
Since version 69, setuptools preserves
_
in Name metadata and.egg-link
files (before it converted them to-
).This is an attempt to make pip freeze and pip uninstall compatible with that behaviour, as well as fix various part of the testsuite that depended on that. The changes should be compatible with previous setuptools versions too.
This includes and extends #12458.