Skip to content

Fix: Handle package.workspace key in child Cargo.toml #3442

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Synss
Copy link

@Synss Synss commented May 15, 2025

Setting the package.workspace key in child workspace fails splicing with an error similar to

Found manifest at /home/mlaurin/src/check_mk.git/packages/site/check-http/Cargo.toml which is a member of the workspace at
/home/mlaurin/src/check_mk.git/packages/site/check-http/.. which isn't included in the crates_universe

Now, the key is valid in Cargo.toml when the code is organized in workspaces. It is even required for out-of-tree workspaces. See https://github.com/nox/rust-rfcs/blob/master/text/1525-cargo-workspace.md

Before December, the key was ignored but would not fail splicing. The regression is most likely due to changes related to #3090

Setting the `package.workspace` key in child workspace fails splicing
with an error similar to

```
Found manifest at /home/mlaurin/src/check_mk.git/packages/site/check-http/Cargo.toml which is a member of the workspace at
/home/mlaurin/src/check_mk.git/packages/site/check-http/.. which isn't included in the crates_universe
```

Now, the key is valid in Cargo.toml when the code is organized in
workspaces.  It is even required for out-of-tree workspaces.  See
https://github.com/nox/rust-rfcs/blob/master/text/1525-cargo-workspace.md

Before December, the key was ignored but would not fail splicing.  The
regression is most likely due to changes related to
bazelbuild#3090
Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Had a few questions for you!

Synss added 2 commits May 22, 2025 16:24
Library to normalize path without symlink resolution.
The trivial `PathCleanUtf8` trait mimics the `Clean` trait implemented
for `PathBuf` in the `clean_path` library.

https://docs.rs/clean-path/latest/clean_path/trait.Clean.html
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.

2 participants