Merge pull request 'fix: error message should include pathname' (#29) from liw/fix into main

Reviewed-on: https://codeberg.org/ambient/ambient-run/pulls/29

1 files changed, 4 insertions, 4 deletions

diff --git a/src/vdrive.rs b/src/vdrive.rs
index 12ec823..03f30f2 100644
--- a/src/vdrive.rs
+++ b/src/vdrive.rs
@@ -91,12 +91,12 @@ impl VirtualDriveBuilder {
         if let Some(root) = self.root {
             builder
                 .append_dir_all(".", &root)
-                .map_err(VirtualDriveError::CreateTar)?;
+                .map_err(|e| VirtualDriveError::CreateTar(root.clone(), e))?;
         }
         if let Some((path, name)) = extra {
             builder
                 .append_path_with_name(path, name)
-                .map_err(VirtualDriveError::CreateTar)?;
+                .map_err(|e| VirtualDriveError::CreateTar(path.to_path_buf(), e))?;
         }
         Ok(VirtualDrive { filename })
     }
@@ -115,8 +115,8 @@ pub enum VirtualDriveError {
     #[error("failed to create virtual drive {0}")]
     Create(PathBuf, #[source] std::io::Error),
 
-    #[error("failed to create tar archive for virtual drive")]
-    CreateTar(#[source] std::io::Error),
+    #[error("failed to create tar archive for virtual drive from {0}")]
+    CreateTar(PathBuf, #[source] std::io::Error),
 
     #[error("failed to open virtual drive {0}")]
     Open(PathBuf, #[source] std::io::Error),