Can't `cargo publish --workspace --dry-run` if the versions already exist

[epage]

Problem

In testing #1169 on cargo release now that #14721 is fixed, I came across another way you can't dry-run existing versions: the overlay registry blocks it

[2024-11-06T21:50:26Z TRACE cargo_release::ops::cmd] /home/epage/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/b
in/cargo publish --manifest-path /home/epage/src/personal/clap/clap_lex/Cargo.toml -Zpackage-workspace --package clap_
lex --package clap_builder --package clap_derive --package clap --package clap_complete --package clap_complete_fig --
package clap_complete_nushell --package clap_mangen --dry-run --allow-dirty
    Updating crates.io index
warning: crate clap@4.5.20 already exists on crates.io index
warning: crate clap_builder@4.5.20 already exists on crates.io index
warning: crate clap_lex@0.7.2 already exists on crates.io index
warning: crate clap_derive@4.5.18 already exists on crates.io index
warning: crate clap_complete@4.5.37 already exists on crates.io index
warning: crate clap_complete_fig@4.5.2 already exists on crates.io index
warning: crate clap_complete_nushell@4.5.4 already exists on crates.io index
warning: crate clap_mangen@0.2.24 already exists on crates.io index
   Packaging clap_lex v0.7.2 (/home/epage/src/personal/clap/clap_lex)
warning: ignoring test `testsuite` as `tests/testsuite/main.rs` is not included in the published package
    Packaged 8 files, 42.9KiB (12.5KiB compressed)
   Packaging clap_builder v4.5.20 (/home/epage/src/personal/clap/clap_builder)
    Packaged 62 files, 914.9KiB (160.0KiB compressed)
   Packaging clap_derive v4.5.18 (/home/epage/src/personal/clap/clap_derive)
    Packaged 22 files, 162.1KiB (29.4KiB compressed)
   Packaging clap v4.5.20 (/home/epage/src/personal/clap)
warning: ignoring test `builder` as `tests/builder/main.rs` is not included in the published package
warning: ignoring test `derive` as `tests/derive/main.rs` is not included in the published package
warning: ignoring test `derive_ui` as `tests/derive_ui.rs` is not included in the published package
warning: ignoring test `examples` as `tests/examples.rs` is not included in the published package
warning: ignoring test `macros` as `tests/macros.rs` is not included in the published package
warning: ignoring test `ui` as `tests/ui.rs` is not included in the published package
    Updating crates.io index
error: failed to prepare local package for uploading

Caused by:
  failed to get `clap_builder` as a dependency of package `clap v4.5.20 (/home/epage/src/personal/clap)`

Caused by:
  found a package in the remote registry and the local overlay: clap_builder@4.5.20

Steps

$ cargo install cargo-release
$ git clone git@github.com:clap-rs/clap.git
$ cd clap
$ cargo +nightly publish -Zpackage-workspace --package clap_
lex --package clap_builder --package clap_derive --package clap --package clap_complete --package clap_complete_fig --
package clap_complete_nushell --package clap_mangen --dry-run --allow-dirty

Possible Solution(s)

Remove the restriction on the overlay which comes at the risk of it being abused.

Notes

No response

Version

cargo 1.84.0-nightly (0310497 2024-11-01)

[epage]

In addition to being accepted, wanted to record some notes from the team's discussion

• We considered making this a lint or flag but those aren't generally dynamic which this would need. Granted, --ignore-rust-version is kind of like that.
• While we want to be cautious about what behavior we allow for this Source, this is still all internal, so it should be good.
• The current error is more strict than the single-package case, so removing the error brings them inline with each other
• The important thing is that we warn which we should already get from fix(publish): Downgrade version-exists error to warning on dry-run  #14742