Delete rmeta files for rlibs during cargo clean

Author: alexcrichton

src/cargo/ops/cargo_clean.rs

@@ -2,7 +2,7 @@ use std::collections::HashMap;
 use std::fs;
 use std::path::Path;
 
-use crate::core::compiler::UnitInterner;
+use crate::core::compiler::{UnitInterner, FileFlavor};
 use crate::core::compiler::{BuildConfig, BuildContext, CompileMode, Context, Kind};
 use crate::core::profiles::UnitFor;
 use crate::core::Workspace;
@@ -119,6 +119,9 @@ pub fn clean(ws: &Workspace<'_>, opts: &CleanOptions<'_>) -> CargoResult<()> {
             if let Some(ref dst) = output.hardlink {
                 rm_rf(dst, config)?;
             }
+            if let FileFlavor::Linkable { rmeta } = &output.flavor {
+                rm_rf(rmeta, config)?;
+            }
         }
     }
 

tests/testsuite/clean.rs

@@ -294,3 +294,26 @@ fn clean_verbose() {
         .run();
     p.cargo("build").run();
 }
+
+#[test]
+fn clean_remove_rlib_rmeta() {
+    let p = project()
+        .file(
+            "Cargo.toml",
+            r#"
+                [package]
+                name = "foo"
+                version = "0.0.1"
+            "#,
+        )
+        .file("src/lib.rs", "")
+        .build();
+
+    p.cargo("build").run();
+    assert!(p.target_debug_dir().join("libfoo.rlib").exists());
+    let rmeta = p.glob("target/debug/deps/*.rmeta").next().unwrap().unwrap();
+    assert!(rmeta.exists());
+    p.cargo("clean -p foo").run();
+    assert!(!p.target_debug_dir().join("libfoo.rlib").exists());
+    assert!(!rmeta.exists());
+}