give better context than an simple string

Author: Skgland

src/crates/git.rs

@@ -86,7 +86,10 @@ impl CrateTrait for GitRepo {
                 .cd(&path)
                 .process_lines(&mut detect_private_repositories)
                 .run()
-                .with_context(|| format!("failed to update {}", self.url))
+                .with_context(|| PrepareError::GitFailure {
+                    action: "update",
+                    url: self.url.clone(),
+                })
         } else {
             info!("cloning repository {}", self.url);
             Command::new(workspace, "git")
@@ -95,7 +98,10 @@ impl CrateTrait for GitRepo {
                 .args(&[&path])
                 .process_lines(&mut detect_private_repositories)
                 .run()
-                .with_context(|| format!("failed to clone {}", self.url))
+                .with_context(|| PrepareError::GitFailure {
+                    action: "clone",
+                    url: self.url.clone(),
+                })
         };
 
         if private_repository && res.is_err() {
@@ -118,7 +124,10 @@ impl CrateTrait for GitRepo {
             .args(&["clone"])
             .args(&[self.cached_path(workspace).as_path(), dest])
             .run()
-            .with_context(|| format!("failed to checkout {}", self.url))?;
+            .with_context(|| PrepareError::GitFailure {
+                action: "checkout",
+                url: self.url.clone(),
+            })?;
         Ok(())
     }
 }

src/prepare.rs

@@ -410,6 +410,14 @@ pub enum PrepareError {
     /// cargo rejected the lockfile
     #[error("the crate has a broken lockfile: \n\n{0}")]
     InvalidCargoLock(String),
+    /// git clone/fetch failed, but repository didn't appear to be private
+    #[error("git failed to {action} for {url}")]
+    GitFailure {
+        /// action that was attempted
+        action: &'static str,
+        /// repo url
+        url: String,
+    },
 }
 
 #[cfg(test)]