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

dependency on a local workspace crate requires full crates path instead of just workspace root #12161

Closed
softstream-link opened this issue May 19, 2023 · 2 comments
Labels
A-workspaces Area: workspaces C-feature-request Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted`

Comments

@softstream-link
Copy link

Problem

project_a
   Cargo.toml # workspace
   crate_1
      Cargo.toml 
   crate_2
      Cargo.toml
project_b
   Cargo.toml
      [dependencies]
      crate_1 = { path = "../project_a/crate_1" }  # Why does this work
      vs
      crate_1 = { path = "../project_a/" }  # And this DOES Not

Proposed Solution

No response

Notes

No response

@softstream-link softstream-link added C-feature-request Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted` S-triage Status: This issue is waiting on initial triage. labels May 19, 2023
@weihanglo
Copy link
Member

Hi! Workspaces cannot recognize each other's member as of now. I'd like to learn the rationale behind why you want to achieve that and how it could be better if Cargo supports that. We have been collecting some real world nested workspace needs for a while. This sounds like a thing the nested workspace feature can resolve. Close in favor of #5042, and feel free to provide feedback!

@weihanglo weihanglo closed this as not planned Won't fix, can't repro, duplicate, stale May 20, 2023
@weihanglo weihanglo added A-workspaces Area: workspaces S-needs-info Status: Needs more info, such as a reproduction or more background for a feature request. and removed S-triage Status: This issue is waiting on initial triage. labels May 20, 2023
@weihanglo
Copy link
Member

BTW, there is a draft proposal for nested workspaces still brewing: https://hackmd.io/AWv4c8dUT9ahWEo4epNG4g.

@weihanglo weihanglo removed the S-needs-info Status: Needs more info, such as a reproduction or more background for a feature request. label May 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-workspaces Area: workspaces C-feature-request Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted`
Projects
None yet
Development

No branches or pull requests

2 participants