Directly build tokio command instead of converting

Author: julianschuler

crates/repository/src/cargo.rs

@@ -4,14 +4,14 @@ use std::{
     ffi::{OsStr, OsString},
     fmt::{Display, Formatter},
     path::Path,
-    process::Command,
 };
 
 use color_eyre::{
     eyre::{bail, Context, ContextCompat},
     Result,
 };
 use pathdiff::diff_paths;
+use tokio::process::Command;
 
 use crate::{data_home::get_data_home, sdk::download_and_install};
 
@@ -73,14 +73,13 @@ impl Cargo {
                 Host::Remote => {
                     let mut command =
                         Command::new(repository_root.as_ref().join("scripts/remoteWorkspace"));
-                    command
+
+                    let status = command
                         .arg("pepsi")
                         .arg("sdk")
                         .arg("install")
                         .arg("--version")
-                        .arg(version);
-
-                    let status = tokio::process::Command::from(command)
+                        .arg(version)
                         .status()
                         .await
                         .wrap_err("failed to run pepsi")?;

tools/pepsi/src/cargo.rs

@@ -112,13 +112,13 @@ pub async fn cargo<CargoArguments: Args + CargoCommand>(
 
     arguments.cargo.apply(&mut cargo);
 
-    let cargo_command = cargo
+    let mut cargo_command = cargo
         .command(&repository_root, compiler_artifacts)
         .wrap_err("failed to create cargo command")?;
 
     debug!("Running `{cargo_command:?}`");
 
-    let status = tokio::process::Command::from(cargo_command)
+    let status = cargo_command
         .status()
         .await
         .wrap_err("failed to run cargo")?;