fix(check): Fixed incorrect locking logic when artifact-dir == build-dir

Author: ranger-ross

src/cargo/core/compiler/layout.rs

@@ -151,7 +151,12 @@ impl Layout {
         // actual destination (sub)subdirectory.
         paths::create_dir_all(dest.as_path_unlocked())?;
 
-        let build_dir_lock = if root == build_root || is_on_nfs_mount(build_root.as_path_unlocked())
+        // We always need to take the build-dir lock but if the build-dir == artifact-dir then we
+        // only take the artifact-dir. (locking both as they are the same dir)
+        // However we need to take into account that for some builds like `cargo check` we avoid
+        // locking the artifact-dir. We still need to lock the build-dir to avoid file corruption.
+        let build_dir_lock = if (must_take_artifact_dir_lock && root == build_root)
+            || is_on_nfs_mount(build_root.as_path_unlocked())
         {
             None
         } else {

tests/testsuite/check.rs

@@ -1710,13 +1710,13 @@ fn check_build_should_not_output_files_to_artifact_dir() {
 }
 
 #[cargo_test]
-fn check_build_should_not_lock_artifact_dir() {
+fn check_build_should_lock_target_dir_when_artifact_dir_is_same_as_build_dir() {
     let p = project()
         .file("src/main.rs", r#"fn main() { println!("Hello, World!") }"#)
         .build();
 
     p.cargo("check").enable_mac_dsym().run();
-    assert!(!p.root().join("target/debug/.cargo-lock").exists());
+    assert!(p.root().join("target/debug/.cargo-lock").exists());
 }
 
 #[cargo_test]