Merge #94

94: Check with which r=matthiasbeyer a=matthiasbeyer

Closes #92 

`@Xmoe` could you please have a look whether this is something you'd like to see here (of course also others are welcome to comment)?!

Co-authored-by: Matthias Beyer <mail@beyermatthias.de>

Author: bors[bot]

.github/workflows/ci.yml

@@ -32,6 +32,7 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: check
+          args: --all-features
 
 
   test:

Cargo.toml

@@ -19,5 +19,11 @@ regex = "1"
 tempfile = "3"
 thiserror = "1.0.34"
 
+which = { version = "4.4", optional = true }
+
 [badges]
 maintenance = { status = "passively-maintained" }
+
+[features]
+default = []
+which = ["dep:which"]

src/error.rs

@@ -39,4 +39,8 @@ pub enum Error {
 
     #[error("The provided program arguments cannot be parsed")]
     BadProgramArguments,
+
+    #[cfg(feature = "which")]
+    #[error(transparent)]
+    Which(#[from] which::Error),
 }

src/session.rs

@@ -232,6 +232,10 @@ pub fn spawn(program: &str, timeout_ms: Option<u64>) -> Result<PtySession, Error
 
 /// See `spawn`
 pub fn spawn_command(command: Command, timeout_ms: Option<u64>) -> Result<PtySession, Error> {
+    #[cfg(feature = "which")]
+    {
+        let _ = which::which(command.get_program())?;
+    }
     let mut process = PtyProcess::new(command)?;
     process.set_kill_timeout(timeout_ms);