Use CommandGroup to let OS kill spawned processes

LPGhatguy · Jan 7, 2022 · 1842a95 · 1842a95
1 parent 26ca90f
commit 1842a95
Show file tree

Hide file tree

Showing 3 changed files with 40 additions and 5 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -11,6 +11,7 @@ repository = "https://github.com/LPGhatguy/aftman"
 [dependencies]
 anyhow = "1.0.43"
 atty = "0.2.14"
+command-group = "1.0.8"
 dialoguer = "0.9.0"
 dirs = "3.0.2"
 env_logger = "0.9.0"

diff --git a/src/tool_storage.rs b/src/tool_storage.rs
@@ -8,6 +8,7 @@ use std::path::{Path, PathBuf};
 use std::process::Command;
 
 use anyhow::{bail, Context};
+use command_group::CommandGroup;
 use fs_err::File;
 use once_cell::unsync::OnceCell;
 
@@ -56,7 +57,7 @@ impl ToolStorage {
         self.install_exact(id)?;
 
         let exe_path = self.exe_path(id);
-        let status = Command::new(exe_path).args(args).status().unwrap();
+        let status = Command::new(exe_path).args(args).group_status().unwrap();
 
         Ok(status.code().unwrap_or(1))
     }