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

Don't read metadata from stale .egg-info files #9760

Merged
merged 1 commit into from
Dec 10, 2024
Merged

Conversation

charliermarsh
Copy link
Member

Summary

We were reading an .egg-info file from the root directory that didn't apply to the root member -- it was for another workspace member. I think this is driven from some idiosyncracies in the setuptools setup for that workspace member, but it's still wrong to fail.

This PR adds a few measures to fix this:

  1. We validate the egg-info filename against the package metadata.
  2. We skip, rather than fail, if we see incorrect metadata in an egg-info file or similar. This is an optimization anyway; worst case, we try to build the package, then fail there.

Closes #9743.

@charliermarsh charliermarsh added the bug Something isn't working label Dec 10, 2024
@charliermarsh
Copy link
Member Author

Need to add a test here.

@charliermarsh charliermarsh enabled auto-merge (squash) December 10, 2024 02:13
@charliermarsh charliermarsh merged commit 8a2e3a8 into main Dec 10, 2024
64 checks passed
@charliermarsh charliermarsh deleted the charlie/egg branch December 10, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

uv pip install not working with #subdirectory in monorepo
1 participant