Deduplicate main error printing

As part of the tracing-subscriber CVE I did a quick audit
for usages of `tracing::error!` and I noticed when
we updated the `main()` function in the primary
crate we missed also doing the same for system-reinstall-bootc.

Move the handling of that to utils.

xref: https://bugzilla.redhat.com/show_bug.cgi?id=2392017
Signed-off-by: Colin Walters <[email protected]>

Author: cgwalters

Cargo.lock

@@ -22,7 +22,6 @@ bootc-utils = { package = "bootc-internal-utils", path = "../utils", version = "
 anstream = { workspace = true }
 anyhow = { workspace = true }
 log = { workspace = true }
-owo-colors = { workspace = true }
 tokio = { workspace = true, features = ["macros"] }
 tracing = { workspace = true }
 

crates/cli/Cargo.toml

@@ -35,15 +35,5 @@ fn run() -> Result<()> {
 }
 
 fn main() {
-    use owo_colors::OwoColorize;
-
-    // In order to print the error in a custom format (with :#) our
-    // main simply invokes a run() where all the work is done.
-    // This code just captures any errors.
-    if let Err(e) = run() {
-        let mut stderr = anstream::stderr();
-        // Don't panic if writing fails
-        let _ = writeln!(stderr, "{}{:#}", "error: ".red(), e);
-        std::process::exit(1);
-    }
+    bootc_utils::run_main(run)
 }

crates/cli/src/main.rs

@@ -19,6 +19,7 @@ bootc-mount = { path = "../mount" }
 bootc-utils = { package = "bootc-internal-utils", path = "../utils", version = "0.0.0" }
 
 # Workspace dependencies
+anstream = { workspace = true }
 anyhow = { workspace = true }
 clap = { workspace = true, features = ["derive"] }
 indoc = { workspace = true }

crates/system-reinstall-bootc/Cargo.toml

@@ -66,11 +66,5 @@ fn run() -> Result<()> {
 }
 
 fn main() {
-    // In order to print the error in a custom format (with :#) our
-    // main simply invokes a run() where all the work is done.
-    // This code just captures any errors.
-    if let Err(e) = run() {
-        tracing::error!("{:#}", e);
-        std::process::exit(1);
-    }
+    bootc_utils::run_main(run)
 }

crates/system-reinstall-bootc/src/main.rs

@@ -8,8 +8,10 @@ repository = "https://github.com/bootc-dev/bootc"
 
 [dependencies]
 # Workspace dependencies
+anstream = { workspace = true }
 anyhow = { workspace = true }
 chrono = { workspace = true, features = ["std"] }
+owo-colors = { workspace = true }
 rustix = { workspace = true }
 serde = { workspace = true, features = ["derive"] }
 serde_json = { workspace = true }

crates/utils/Cargo.toml

@@ -16,3 +16,23 @@ pub use tracing_util::*;
 pub mod reexec;
 mod result_ext;
 pub use result_ext::*;
+
+/// Intended for use in `main`, calls an inner function and
+/// handles errors by printing them.
+pub fn run_main<F>(f: F)
+where
+    F: FnOnce() -> anyhow::Result<()>,
+{
+    use std::io::Write as _;
+
+    use owo_colors::OwoColorize;
+
+    // NOTE: The main logic in crates/cli/main.rs is the canonical
+    // copy of this.
+    if let Err(e) = f() {
+        let mut stderr = anstream::stderr();
+        // Don't panic if writing fails
+        let _ = writeln!(stderr, "{}{:#}", "error: ".red(), e);
+        std::process::exit(1);
+    }
+}