utils: Don't use run in run_with_cmd_context

`run` hides stderr until the command fails, which is not what we want
for all current users of `run_with_cmd_context` - it's currently used
by long running processes where we want the users to see all the output
live.

Signed-off-by: Omer Tuchfeld <[email protected]>

Author: omertuc

utils/src/command.rs

@@ -139,7 +139,9 @@ impl CommandRunExt for Command {
     }
 
     fn run_with_cmd_context(&mut self) -> Result<()> {
-        self.run()
+        self.status()?
+            .success()
+            .then_some(())
             // The [`Debug`] output of command contains a properly shell-escaped commandline
             // representation that the user can copy paste into their shell
             .context("Failed to run command: {self:#?}")