Refactor and clean up xtask package (#1540)

* Simplify building/running examples, allow building a single example

* Simplify building/running of tests a bit

* Clean up the `run_elfs` function a bit

* Remove unnecessary duplication in `cargo` module

xtask/src/cargo.rs

@@ -22,8 +22,9 @@ pub fn run(args: &[String], cwd: &Path) -> Result<()> {
     let status = Command::new(get_cargo())
         .args(args)
         .current_dir(cwd)
-        .stdout(Stdio::piped())
+        .stdout(Stdio::inherit())
         .stderr(Stdio::inherit())
+        .stdin(Stdio::inherit())
         .status()?;
 
     // Make sure that we return an appropriate exit code here, as Github Actions
@@ -35,27 +36,6 @@ pub fn run(args: &[String], cwd: &Path) -> Result<()> {
     }
 }
 
-/// Execute cargo with the given arguments and from the specified directory.
-pub fn run_with_input(args: &[String], cwd: &Path) -> Result<()> {
-    if !cwd.is_dir() {
-        bail!("The `cwd` argument MUST be a directory");
-    }
-
-    let status = Command::new(get_cargo())
-        .args(args)
-        .current_dir(cwd)
-        .stdout(std::process::Stdio::inherit())
-        .stderr(std::process::Stdio::inherit())
-        .stdin(std::process::Stdio::inherit())
-        .status()?;
-
-    if status.success() {
-        Ok(())
-    } else {
-        Err(anyhow::anyhow!("Failed to execute cargo subcommand"))
-    }
-}
-
 fn get_cargo() -> String {
     #[cfg(target_os = "windows")]
     let cargo = if let Ok(cargo) = std::env::var("CARGO_HOME") {

xtask/src/lib.rs

@@ -294,7 +294,7 @@ pub fn execute_app(
     let args = builder.build();
     log::debug!("{args:#?}");
 
-    cargo::run_with_input(&args, package_path)
+    cargo::run(&args, package_path)
 }
 
 /// Build the specified package, using the given toolchain/target/features if