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

Stop workspace discovery for git and other cache entries at cache entry root #3936

Closed
konstin opened this issue May 31, 2024 · 1 comment · Fixed by #8665
Closed

Stop workspace discovery for git and other cache entries at cache entry root #3936

konstin opened this issue May 31, 2024 · 1 comment · Fixed by #8665

Comments

@konstin
Copy link
Member

konstin commented May 31, 2024

Currently, workspace discovery walks from the project pyproject.toml to the file system root to find a workspace pyproject.toml. For cache entries such as git checkout and temporary directories, we should only walk up to the cache entry boundary; we don't want to consider any pyproject.toml files the user may have above the cache root.

@konstin konstin added the preview Experimental behavior label May 31, 2024
@charliermarsh
Copy link
Member

What are Cargo's rules here?

@zanieb zanieb removed the preview Experimental behavior label Aug 20, 2024
konstin added a commit that referenced this issue Oct 29, 2024
When resolving workspace dependencies from a workspace that's in git, we need to convert these transitive dependencies to git dependencies. This fixes a bug where we would treat them as path dependencies inside the checkout directory, leading either to clashes (between a local path and another direct git dependency) or invalid lockfiles (referencing the checkout dir).

Fixes #8087
Fixes #4920
Fixes #3936 since we needed that information anyway
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 a pull request may close this issue.

3 participants