[fix] fixed the bug where BashUtil.RemoveDir failed to run under certain circumstances on macOS systems.

pirunxi · pirunxi · commit 8d1280a1bd99 · 2025-03-14T08:59:11.000+08:00
diff --git a/Editor/Installer/BashUtil.cs b/Editor/Installer/BashUtil.cs
@@ -83,7 +83,7 @@ public static void RemoveDir(string dir, bool log = false)
                     {
                         RemoveDir(subDir);
                     }
-                    Directory.Delete(dir);
+                    Directory.Delete(dir, true);
                     break;
                 }
                 catch (Exception e)
@@ -111,12 +111,12 @@ private static void CopyWithCheckLongFile(string srcFile, string dstFile)
 #endif
             if (srcFile.Length > maxPathLength)
             {
-                UnityEngine.Debug.LogError($"srcFile:{srcFile} path is too long. copy ignore!");
+                UnityEngine.Debug.LogError($"srcFile:{srcFile} path is too long. skip copy!");
                 return;
             }
             if (dstFile.Length > maxPathLength)
             {
-                UnityEngine.Debug.LogError($"dstFile:{dstFile} path is too long. copy ignore!");
+                UnityEngine.Debug.LogError($"dstFile:{dstFile} path is too long. skip copy!");
                 return;
             }
             File.Copy(srcFile, dstFile);

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ public static void RemoveDir(string dir, bool log = false)`
`83`	`83`	`{`
`84`	`84`	`RemoveDir(subDir);`
`85`	`85`	`}`
`86`		`- Directory.Delete(dir);`
	`86`	`+ Directory.Delete(dir, true);`
`87`	`87`	`break;`
`88`	`88`	`}`
`89`	`89`	`catch (Exception e)`
`@@ -111,12 +111,12 @@ private static void CopyWithCheckLongFile(string srcFile, string dstFile)`
`111`	`111`	`#endif`
`112`	`112`	`if (srcFile.Length > maxPathLength)`
`113`	`113`	`{`
`114`		`- UnityEngine.Debug.LogError($"srcFile:{srcFile} path is too long. copy ignore!");`
	`114`	`+ UnityEngine.Debug.LogError($"srcFile:{srcFile} path is too long. skip copy!");`
`115`	`115`	`return;`
`116`	`116`	`}`
`117`	`117`	`if (dstFile.Length > maxPathLength)`
`118`	`118`	`{`
`119`		`- UnityEngine.Debug.LogError($"dstFile:{dstFile} path is too long. copy ignore!");`
	`119`	`+ UnityEngine.Debug.LogError($"dstFile:{dstFile} path is too long. skip copy!");`
`120`	`120`	`return;`
`121`	`121`	`}`
`122`	`122`	`File.Copy(srcFile, dstFile);`