Merge #61

61: fix run ignoring stdin r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.2.3
+
+- Fix bug where `Cmd::run` would ignore specified stdin.
+
 ## 0.2.2
 
 - Add `Shell::path_exists`.

Cargo.toml

@@ -2,7 +2,7 @@
 name = "xshell"
 description = "Utilities for quick shell scripting in Rust"
 categories = ["development-tools::build-utils", "filesystem"]
-version = "0.2.2" # also update xshell-macros/Cargo.toml and CHANGELOG.md
+version = "0.2.3" # also update xshell-macros/Cargo.toml and CHANGELOG.md
 license = "MIT OR Apache-2.0"
 repository = "https://github.com/matklad/xshell"
 authors = ["Aleksey Kladov <aleksey.kladov@gmail.com>"]
@@ -14,7 +14,7 @@ exclude = [".github/", "bors.toml", "rustfmt.toml", "cbench", "mock_bin/"]
 [workspace]
 
 [dependencies]
-xshell-macros = { version = "=0.2.2", path = "./xshell-macros" }
+xshell-macros = { version = "=0.2.3", path = "./xshell-macros" }
 
 [dev-dependencies]
 anyhow = "1.0.56"

src/lib.rs

@@ -960,10 +960,7 @@ impl<'a> Cmd<'a> {
         if !self.data.quiet {
             eprintln!("$ {}", self);
         }
-        let mut command = self.to_command();
-        let status = command.status().map_err(|err| Error::new_cmd_io(self, err))?;
-        self.check_status(status)?;
-        Ok(())
+        self.output_impl(false, false).map(|_| ())
     }
 
     /// Run the command and return its stdout as a string.

xshell-macros/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "xshell-macros"
 description = "Private implementation detail of xshell crate"
-version = "0.2.2"
+version = "0.2.3"
 license = "MIT OR Apache-2.0"
 repository = "https://github.com/matklad/xshell"
 authors = ["Aleksey Kladov <aleksey.kladov@gmail.com>"]