Adding a `registry` and `version` to a local dependency causes dependents over Git to attempt to pull the registry by name

[w4]

Problem

When moving crates to registries that were previously depended on via Git, adding a registry definition to a local dependency causes dependents to attempt to pull the registry, rather than ignoring it and using the local path.

Steps

1. crate-a containing:

   $ cat Cargo.toml
   [package]
   name = "crate-a"
   version = "0.1.0"
   edition = "2021"
   
   [dependencies]
   some-crate = { path = "../some-crate", registry = "x", version = "0.1" }
   

2. crate-b containing:

   $ cat Cargo.toml
   [package]
   name = "crate-b"
   version = "0.1.0"
   edition = "2021"
   
   [dependencies]
   crate-a = { git = "https://git.x.com/path/to/crate-a" }
   

3. Attempting to compile crate-b attempts to pull the x registry rather than ignoring it since it is overridden by the relative path.

When combined with #12271, this prevents any sort of slow migration to registries -- requiring a big bang update to all crates which is untenable for any mid to large sized organisation.

Possible Solution(s)

When depending on a crate using { git = "https://git.x.com/path/to/repo" }, the local dependency should be preferred over the one in the registry.

Notes

No response

Version

No response

[weihanglo]

Wrong thread. Just deleted the comment. Sorry for the noise.

[weihanglo]

Copied from #12271 (comment), since this is the right place for the comment.

---

Note that currently if you have both path and registry declared in your direct dependency and the registry config is missing, Cargo will error out and stop the build.

There are several scenario of them, and we may only need to change the behavior for git and path dependencies if we really want it.

• ✅ direct dependency -> transitive dependency with both path and registry: shouldn't exist on crates.io. No change needed.
• ✅ direct registry dependency -> transitive dependency with both path and registry: Cargo will stick to registry dependencies transitively. No change needed.
• 🤯 direct git dependency -> transitive dependency with both path and registry: Cargo will use path dependency from the same git repo of the direct git dependency. Currently it errors out as registry cannot be found. This seems to have chance to relax a bit.
• ❓ direct path dependency -> transitive dependency with both path and registry: Cargo will favor path dependencies for transitive path dependencies. It's a bit odd to relax the check for this case, since usually you have the control over your local path dependencies.