fix(vendor): downgrade package cache lock to DownloadExclusive

weihanglo · weihanglo · commit b2fbd65a1382 · 2025-05-20T17:40:36.000-04:00
It is safe since `cargo vendor` no longer deletes any files.
diff --git a/src/cargo/ops/vendor.rs b/src/cargo/ops/vendor.rs
@@ -39,7 +39,7 @@ pub fn vendor(ws: &Workspace<'_>, opts: &VendorOptions<'_>) -> CargoResult<()> {
         extra_workspaces.push(ws);
     }
     let workspaces = extra_workspaces.iter().chain(Some(ws)).collect::<Vec<_>>();
-    let _lock = gctx.acquire_package_cache_lock(CacheLockMode::MutateExclusive)?;
+    let _lock = gctx.acquire_package_cache_lock(CacheLockMode::DownloadExclusive)?;
     let vendor_config = sync(gctx, &workspaces, opts).context("failed to sync")?;
 
     if gctx.shell().verbosity() != Verbosity::Quiet {

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ pub fn vendor(ws: &Workspace<'_>, opts: &VendorOptions<'_>) -> CargoResult<()> {`
`39`	`39`	`extra_workspaces.push(ws);`
`40`	`40`	`}`
`41`	`41`	`let workspaces = extra_workspaces.iter().chain(Some(ws)).collect::<Vec<_>>();`
`42`		`- let _lock = gctx.acquire_package_cache_lock(CacheLockMode::MutateExclusive)?;`
	`42`	`+ let _lock = gctx.acquire_package_cache_lock(CacheLockMode::DownloadExclusive)?;`
`43`	`43`	`let vendor_config = sync(gctx, &workspaces, opts).context("failed to sync")?;`
`44`	`44`
`45`	`45`	`if gctx.shell().verbosity() != Verbosity::Quiet {`