Skip to content

deduplicate workspace package names #427

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

Merged
merged 6 commits into from
Jun 22, 2024
Merged

deduplicate workspace package names #427

merged 6 commits into from
Jun 22, 2024

Conversation

OmarTawfik
Copy link
Contributor

@OmarTawfik OmarTawfik commented Jun 22, 2024

Currently the recommended way to install bencher_cli from source produces the following warnings:

$ cargo install bencher_cli --git https://github.com/bencherdev/bencher --branch main

    Updating git repository `https://github.com/bencherdev/bencher`
warning: skipping duplicate package `game` found at `/.../rust/git/checkouts/bencher-c9d027409129568d/6b48e62/examples/rust/iai`
warning: skipping duplicate package `game` found at `/.../rust/git/checkouts/bencher-c9d027409129568d/6b48e62/examples/rust/criterion`
warning: skipping duplicate package `xtask` found at `/.../rust/git/checkouts/bencher-c9d027409129568d/6b48e62/examples/ebpf/xdp/xtask`

This deduplicates the different package names in the workspace by adding parent-specific prefixes.

@OmarTawfik OmarTawfik marked this pull request as ready for review June 22, 2024 02:05
@OmarTawfik OmarTawfik marked this pull request as draft June 22, 2024 02:08
@OmarTawfik OmarTawfik marked this pull request as ready for review June 22, 2024 02:19
@epompeii epompeii changed the base branch from main to devel June 22, 2024 10:22
@epompeii
Copy link
Member

Thank you, @OmarTawfik for pointing this out!

This seems to be a false positive. The examples directory is not in the workspace.
There is an outstanding issue to fix this: rust-lang/cargo#10752

In the meantime, the suggested workaround seems to be setting publish = false.

@epompeii epompeii merged commit 9f68493 into bencherdev:devel Jun 22, 2024
5 of 40 checks passed
@OmarTawfik OmarTawfik deleted the fix-dups branch June 24, 2024 06:43
@OmarTawfik OmarTawfik restored the fix-dups branch June 24, 2024 06:43
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