fix(cli.rs): before dev process kill, closes #1626 (#1700)

tauri-apps · May 4, 2021 · ac2cbcb · ac2cbcb
1 parent 2b4e2b7
commit ac2cbcb
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/.changes/fix-before-dev-command-kill.md b/.changes/fix-before-dev-command-kill.md
@@ -0,0 +1,5 @@
+---
+"cli.rs": patch
+---
+
+Properly kill `beforeDevCommand` process.
diff --git a/tooling/cli.rs/src/dev.rs b/tooling/cli.rs/src/dev.rs
@@ -29,7 +29,18 @@ static BEFORE_DEV: OnceCell<Mutex<Child>> = OnceCell::new();
 
 fn kill_before_dev_process() {
   if let Some(child) = BEFORE_DEV.get() {
-    let _ = child.lock().unwrap().kill();
+    let mut child = child.lock().unwrap();
+    #[cfg(windows)]
+    let _ = Command::new("powershell")
+      .arg("-Command")
+      .arg(format!("function Kill-Tree {{ Param([int]$ppid); Get-CimInstance Win32_Process | Where-Object {{ $_.ParentProcessId -eq $ppid }} | ForEach-Object {{ Kill-Tree $_.ProcessId }}; Stop-Process -Id $ppid }}; Kill-Tree {}", child.id()))
+      .status();
+    #[cfg(not(windows))]
+    let _ = Command::new("pkill")
+      .args(&["-TERM", "-P"])
+      .arg(child.id().to_string())
+      .status();
+    let _ = child.kill();
   }
 }