fix PR

Author: wolfv

crates/rattler/src/install/driver.rs

@@ -1,5 +1,5 @@
 use super::clobber_registry::ClobberRegistry;
-use super::unlink::recursively_remove_empty_directories;
+use super::unlink::{recursively_remove_empty_directories, UnlinkError};
 use super::{InstallError, Transaction};
 use futures::stream::FuturesUnordered;
 use futures::{FutureExt, StreamExt};
@@ -166,7 +166,10 @@ impl InstallDriver {
         let required_packages =
             PackageRecord::sort_topologically(prefix_records.iter().collect::<Vec<_>>());
 
-        self.remove_empty_directories(transaction, &prefix_records, target_prefix)?;
+        self.remove_empty_directories(transaction, &prefix_records, target_prefix)
+            .unwrap_or_else(|e| {
+                tracing::warn!("Failed to remove empty directories: {} (ignored)", e);
+            });
 
         self.clobber_registry()
             .unclobber(&required_packages, target_prefix)
@@ -181,7 +184,7 @@ impl InstallDriver {
         transaction: &Transaction<PrefixRecord, RepoDataRecord>,
         new_prefix_records: &[PrefixRecord],
         target_prefix: &Path,
-    ) -> Result<(), InstallError> {
+    ) -> Result<(), UnlinkError> {
         let mut keep_directories = HashSet::new();
 
         // find all forced directories in the prefix records
@@ -218,8 +221,7 @@ impl InstallDriver {
                     target_prefix,
                     is_python_noarch,
                     &keep_directories,
-                )
-                .unwrap();
+                )?;
 
                 // The directory is not empty which means our parent directory is also not empty,
                 // recursively remove the parent directory from the set as well.

crates/rattler/src/install/unlink.rs

@@ -185,10 +185,6 @@ mod tests {
         let prefix_record =
             PrefixRecord::from_repodata_record(repodata_record, None, None, paths, None, None);
 
-        // install_driver
-        //     .post_process(transaction, target_prefix)
-        //     .unwrap();
-
         return prefix_record;
     }