fix: remove symlink on Windows

Signed-off-by: hi-rustin <rustin.liu@gmail.com>

Author: 0xPoe

crates/cargo-util/src/paths.rs

@@ -493,24 +493,53 @@ fn _remove_dir(p: &Path) -> Result<()> {
 ///
 /// If the file is readonly, this will attempt to change the permissions to
 /// force the file to be deleted.
+/// And if the file is a symlink to a directory, this will attempt to remove
+/// the symlink itself.
 pub fn remove_file<P: AsRef<Path>>(p: P) -> Result<()> {
     _remove_file(p.as_ref())
 }
 
 fn _remove_file(p: &Path) -> Result<()> {
-    let mut err = match fs::remove_file(p) {
-        Ok(()) => return Ok(()),
-        Err(e) => e,
-    };
-
-    if err.kind() == io::ErrorKind::PermissionDenied && set_not_readonly(p).unwrap_or(false) {
-        match fs::remove_file(p) {
-            Ok(()) => return Ok(()),
-            Err(e) => err = e,
+    #[cfg(target_os = "windows")]
+    {
+        let metadata = symlink_metadata(p)?;
+        let file_type = metadata.file_type();
+        if file_type.is_symlink() && metadata.is_dir() {
+            return remove_symlink_dir_with_permission_check(p);
         }
     }
 
-    Err(err).with_context(|| format!("failed to remove file `{}`", p.display()))
+    remove_file_with_permission_check(p)
+}
+
+#[cfg(target_os = "windows")]
+fn remove_symlink_dir_with_permission_check(p: &Path) -> Result<()> {
+    remove_with_permission_check(fs::remove_dir, p)
+        .context(format!("failed to remove symlink dir `{}`", p.display()))
+}
+
+fn remove_file_with_permission_check(p: &Path) -> Result<()> {
+    remove_with_permission_check(fs::remove_file, p)
+        .context(format!("failed to remove file `{}`", p.display()))
+}
+
+fn remove_with_permission_check<F, P>(remove_func: F, p: P) -> io::Result<()>
+where
+    F: Fn(P) -> io::Result<()>,
+    P: AsRef<Path> + Clone,
+{
+    match remove_func(p.clone()) {
+        Ok(()) => Ok(()),
+        Err(e) => {
+            if e.kind() == io::ErrorKind::PermissionDenied
+                && set_not_readonly(p.as_ref()).unwrap_or(false)
+            {
+                remove_func(p)
+            } else {
+                Err(e)
+            }
+        }
+    }
 }
 
 fn set_not_readonly(p: &Path) -> io::Result<bool> {
@@ -858,4 +887,27 @@ mod tests {
             );
         }
     }
+
+    #[test]
+    #[cfg(windows)]
+    fn test_remove_symlink_dir() {
+        use super::*;
+        use std::fs;
+        use std::os::windows::fs::symlink_dir;
+
+        let tmpdir = tempfile::tempdir().unwrap();
+        let dir_path = tmpdir.path().join("testdir");
+        let symlink_path = tmpdir.path().join("symlink");
+
+        fs::create_dir(&dir_path).unwrap();
+
+        symlink_dir(&dir_path, &symlink_path).expect("failed to create symlink");
+
+        assert!(symlink_path.exists());
+
+        remove_dir(symlink_path.clone()).unwrap();
+
+        assert!(!symlink_path.exists());
+        assert!(dir_path.exists());
+    }
 }