utils: Use run helper for editor

I just happened to glance at this code, this gives us stderr in the error, etc. Signed-off-by: Colin Walters <[email protected]>
cgwalters · Dec 6, 2024 · 479cbca · 479cbca
1 parent af207a0
commit 479cbca
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/lib/src/utils.rs b/lib/src/utils.rs
@@ -117,16 +117,12 @@ pub(crate) fn spawn_editor(tmpf: &tempfile::NamedTempFile) -> Result<()> {
     let argv0 = editor_args
         .next()
         .ok_or_else(|| anyhow::anyhow!("Invalid editor: {editor}"))?;
-    let status = Command::new(argv0)
+    Command::new(argv0)
         .args(editor_args)
         .arg(tmpf.path())
         .lifecycle_bind()
-        .status()
-        .context("Spawning editor")?;
-    if !status.success() {
-        anyhow::bail!("Invoking editor: {editor} failed: {status:?}");
-    }
-    Ok(())
+        .run()
+        .with_context(|| format!("Invoking editor {editor} failed"))
 }
 
 /// Convert a combination of values (likely from CLI parsing) into a signature source