Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @weihanglo (or someone else) soon.

Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (S-waiting-on-review and S-waiting-on-author) stays updated, invoking these commands when appropriate:

• @rustbot author: the review is finished, PR author should check the comments and take action accordingly
• @rustbot review: the author is ready for a review, this PR will be queued again in the reviewer's queue

While the effort is pretty much appreciated, the current approach doesn't seem to work.

As ehuss mentioned earlier, source replacement doesn't support path dependencies. The current output vendor-config is like

[source."path+file:///home/user/cargo/target/tmp/cit/t0/foo/bar"]
directory = "file:///home/user/cargo/target/tmp/cit/t0/foo/bar"
replace-with = "vendored-sources"

[source.vendored-sources]
directory = "vendor"

It makes bar a directory source at ~/cargo/target/tmp/cit/t0/foo/bar, but the bar is not really a directory source. If you add the config to your .cargo/config.toml and run cargo build --verbose, you'll see that cargo won't pick the vendored bar from vendor/bar.

We'd happy to discuss use cases, workflows and design first in #10134, and see which approach fits better with the community need and the current architecture of Cargo.

☔ The latest upstream changes (presumably #12959) made this pull request unmergeable. Please resolve the merge conflicts.

I am going to close this and keep the alternative #13347 open. #13347 is a proof-of-concept that addresses the issue mentioned in #12858 (comment). It supports path sources in source replacement so that Cargo can pick up correct sources.

Thanks for the pull request!