make xtask use anyhow

Cargo.lock

@@ -4997,6 +4997,7 @@ checksum = "ec7a2a501ed189703dba8b08142f057e887dfc4b2cc4db2d343ac6376ba3e0b9"
 name = "xtask"
 version = "0.28.1"
 dependencies = [
+ "anyhow",
  "log",
 ]
 

xtask/Cargo.toml

@@ -11,3 +11,4 @@ workspace = true
 
 [dependencies]
 log = { workspace = true }
+anyhow = "1.0"

xtask/src/deny.rs

@@ -4,11 +4,9 @@
 
 use std::process::Command;
 
-use super::DynError;
-
-pub fn deny(args: &[&str]) -> Result<(), DynError> {
+pub fn deny(args: &[&str]) -> anyhow::Result<()> {
     if !args.is_empty() {
-        return Err(format!("Invalid arguments: {args:?}").into());
+        anyhow::bail!(format!("Invalid arguments: {args:?}"));
     }
     install_cargo_deny()?;
     let targets = [
@@ -39,13 +37,13 @@ pub fn deny(args: &[&str]) -> Result<(), DynError> {
         super::utils::print_cmd(&cmd);
         let status = cmd.status()?;
         if !status.success() {
-            return Err(status.to_string().into());
+            anyhow::bail!(status.to_string());
         }
     }
     Ok(())
 }
 
-fn install_cargo_deny() -> Result<(), DynError> {
+fn install_cargo_deny() -> anyhow::Result<()> {
     let already_installed = Command::new("cargo")
         .args(["deny", "--version"])
         .output()

xtask/src/main.rs

@@ -7,23 +7,18 @@
 mod deny;
 pub(crate) mod utils;
 
-type DynError = Box<dyn std::error::Error>;
-
-fn main() {
-    if let Err(e) = try_main() {
-        log::error!("{e}");
-        std::process::exit(-1);
-    }
+fn main() -> anyhow::Result<()> {
+    try_main()
 }
 
-fn try_main() -> Result<(), DynError> {
+fn try_main() -> anyhow::Result<()> {
     let arg_strings: Vec<_> = std::env::args().skip(1).collect();
     let args: Vec<_> = arg_strings.iter().map(String::as_str).collect();
 
     match args.as_slice() {
         &[] | &["-h"] | &["--help"] => print_help(),
         &["deny", ..] => deny::deny(&args[1..])?,
-        c => Err(format!("Invalid arguments {c:?}"))?,
+        c => anyhow::bail!(format!("Invalid arguments {c:?}")),
     }
     Ok(())
 }

xtask/src/utils.rs

@@ -4,8 +4,6 @@ use std::{
     process::Command,
 };
 
-use super::DynError;
-
 /// Print the command and its arguments as if the user had typed them
 pub fn print_cmd(cmd: &Command) {
     print!("{} ", cmd.get_program().to_string_lossy());
@@ -18,7 +16,7 @@ pub fn print_cmd(cmd: &Command) {
 /// Prompt user before running a command
 ///
 /// Adapted from [miri](https://github.com/rust-lang/miri/blob/dba35d2be72f4b78343d1a0f0b4737306f310672/cargo-miri/src/util.rs#L181-L204)
-pub fn ask_to_run(mut cmd: Command, ask: bool, reason: &str) -> Result<(), DynError> {
+pub fn ask_to_run(mut cmd: Command, ask: bool, reason: &str) -> anyhow::Result<()> {
     // Disable interactive prompts in CI (GitHub Actions, Travis, AppVeyor, etc).
     // Azure doesn't set `CI` though (nothing to see here, just Microsoft being Microsoft),
     // so we also check their `TF_BUILD`.
@@ -30,16 +28,16 @@ pub fn ask_to_run(mut cmd: Command, ask: bool, reason: &str) -> Result<(), DynEr
         io::stdin().read_line(&mut buf).unwrap();
         match buf.trim().to_lowercase().as_ref() {
             "" | "y" | "yes" => {}
-            "n" | "no" => return Err("Aborting as per your request".into()),
-            a => return Err(format!("Invalid answer `{a}`").into()),
+            "n" | "no" => anyhow::bail!("Aborting as per your request"),
+            a => anyhow::bail!(format!("Invalid answer `{a}`")),
         };
     } else {
         log::info!("Running `{cmd:?}` to {reason}.");
     }
 
     let status = cmd.status()?;
     if !status.success() {
-        return Err(format!("failed to {reason}: {status}").into());
+        anyhow::bail!(format!("failed to {reason}: {status}"));
     }
     Ok(())
 }