parse command output consistently

This commit is contained in:
Remo Senekowitsch
2026-09-29 12:14:52 +02:00
parent cf3a66fe5e
commit b677a2c20f
+5 -6
View File
@@ -22,7 +22,7 @@ pub fn init() -> Result<()> {
bail!(RUSTLINGS_DIR_ALREADY_EXISTS_ERR); bail!(RUSTLINGS_DIR_ALREADY_EXISTS_ERR);
} }
let locate_project_output = Command::new("cargo") let mut locate_project_output = Command::new("cargo")
.arg("locate-project") .arg("locate-project")
.arg("-q") .arg("-q")
.arg("--workspace") .arg("--workspace")
@@ -60,12 +60,11 @@ pub fn init() -> Result<()> {
bail!(IN_INITIALIZED_DIR_ERR); bail!(IN_INITIALIZED_DIR_ERR);
} }
let workspace_manifest = { // Remove newline
let mut stdout = String::try_from(locate_project_output.stdout) locate_project_output.stdout.pop();
let workspace_manifest = String::try_from(locate_project_output.stdout)
.context("Failed to convert the output of `cargo locate-project …` to a string")?; .context("Failed to convert the output of `cargo locate-project …` to a string")?;
stdout.truncate(stdout.trim_end().len()); // trim trailing newline
stdout
};
let workspace_manifest_content = fs::read_to_string(&workspace_manifest) let workspace_manifest_content = fs::read_to_string(&workspace_manifest)
.with_context(|| format!("Failed to read the file {workspace_manifest}"))?; .with_context(|| format!("Failed to read the file {workspace_manifest}"))?;