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

Fix python_version metadata extraction for bdist_egg distributions #1192

Closed
wants to merge 1 commit into from

Conversation

dnicolodi
Copy link
Contributor

Fixes #1190.

@woodruffw
Copy link
Member

This looks fine to me behaviorally, but is there a reason we're supporting bdist_egg here at all, rather than just removing it? Per #1190 (comment) this hasn't been a showstopping bug for any other index (and PyPI doesn't support eggs anymore), so I think the parsimonious thing to do would be just to delete this codepath.

@dnicolodi
Copy link
Contributor Author

I think the fix is much less invasive than removing support for eggs. However, if there is consensus that removing support for eggs is the best way forward, I can do that instead. If support for eggs is removed, support for wininsts should probably be removed too.

@dnicolodi
Copy link
Contributor Author

bdist_egg support has been removed in #1195

@dnicolodi dnicolodi closed this Dec 7, 2024
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

Successfully merging this pull request may close these issues.

python_version for egg binary distributions is wrong
2 participants