Add missing "path context" for more IO failures

Whenever a _user configurable_ path is not found, this currently prints
a useless "File not found" without any additional context _which_ path
was attempted nor where it was configured.  By explaining what we're
doing in the error messages, users are more likely to know what and
where to correct.

Author: MarijnS95

xbuild/src/command/build.rs

@@ -149,7 +149,12 @@ pub fn build(env: &BuildEnv) -> Result<()> {
                                 .package_root()
                                 .join(runtime_lib_path)
                                 .join(target.android_abi().android_abi());
-                            let entries = std::fs::read_dir(abi_dir)?;
+                            let entries = std::fs::read_dir(&abi_dir).with_context(|| {
+                                format!(
+                                    "Runtime libraries for current ABI not found at `{}`",
+                                    abi_dir.display()
+                                )
+                            })?;
                             for entry in entries {
                                 let entry = entry?;
                                 let path = entry.path();

xbuild/src/download.rs

@@ -1,5 +1,5 @@
 use crate::{BuildEnv, Platform};
-use anyhow::Result;
+use anyhow::{Context as _, Result};
 use indicatif::{ProgressBar, ProgressDrawTarget, ProgressStyle};
 use mvn::Download;
 use reqwest::blocking::Client;
@@ -115,11 +115,11 @@ impl<'a> DownloadManager<'a> {
     }
 
     fn rustup_target(&self, target: &str) -> Result<()> {
-        let status = Command::new("rustup")
-            .arg("target")
-            .arg("add")
-            .arg(target)
-            .status()?;
+        let mut command = Command::new("rustup");
+        command.arg("target").arg("add").arg(target);
+        let status = command
+            .status()
+            .with_context(|| format!("Failed to run `{:?}`", command))?;
         anyhow::ensure!(status.success(), "failure running rustup target add");
         Ok(())
     }

xcommon/src/lib.rs

@@ -23,7 +23,10 @@ pub struct Scaler {
 
 impl Scaler {
     pub fn open<P: AsRef<Path>>(path: P) -> Result<Self> {
-        let img = ImageReader::open(path)?.decode()?;
+        let path = path.as_ref();
+        let img = ImageReader::open(path)
+            .with_context(|| format!("Scaler failed to open image at `{}`", path.display()))?
+            .decode()?;
         let (width, height) = img.dimensions();
         anyhow::ensure!(width == height, "expected width == height");
         anyhow::ensure!(width >= 512, "expected icon of at least 512x512 px");