Skip to content

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