-
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
Better error message for missing RECORD file #8954
Comments
Since |
Shouldn't refusing to uninstall/upgrade a package with no |
@uranusjr : could you reconsider the "vendored dependency" label? ISTM that |
You’re right, I misidentified the cause. pip should anticipate the |
I will work on a PR. |
Here: #9949 |
…t-info/RECORD Fixes pypa#8954
What's the problem this feature will solve?
According to PEP 627, the RECORD file is now optional and its absence indicates that a package should not be uninstalled/upgraded by pip or another tool that relies on its content (see the last paragraph in docs). In Fedora, when you install an RPM package without .dist-info/RECORD file and then you try to uninstall it via pip, you'll get a traceback like this:
Describe the solution you'd like
It would be nice to produce a better error message for cases like the one described above. It'd be also nice to use
.dist-info/INSTALLER
file (if exists) to indicate which tool a user should use to work with the package. Note that the INSTALLER file is also optional according to the PEP.Additional context
PEP 627: https://www.python.org/dev/peps/pep-0627/
Related to: #8156
Packaging docs: https://packaging.python.org/specifications/recording-installed-packages/
Cc: @hroncok
The text was updated successfully, but these errors were encountered: