git-cache-proxy: housekeeping: Use eval, not ad-hoc thing involving $ok

diff --git a/scripts/git-cache-proxy b/scripts/git-cache-proxy
index 334f509c5452d57a4dba00a42eebe8f1399a773f..2f38de1154be164d316d7fb06579a22841d521c1 100755 (executable)
--- a/scripts/git-cache-proxy
+++ b/scripts/git-cache-proxy
@@ -405,22 +405,24 @@ sub housekeeping () {
            next;
        }
        logm 'info', "housekeeping: subdirs $subdir: cleaning";
            next;
        }
        logm 'info', "housekeeping: subdirs $subdir: cleaning";

-       my $ok = 1;
-       foreach my $suffix (qw(tmp git)) {
-           my $dir = "${subdir}\\.$suffix";
-           my $errs;
-           remove_tree($dir, { safe=>1, error=>\$errs });
-           if (stat $dir) {
-               $ok = 0;
-               logm 'warning', "housekeeping: $dir: problems with".
-                   " deletion prevent cleanup:";
-               foreach my $err (@$errs) {
-                   my ($file, $message) = %$err;
-                   logm 'info', "problem deleting: $file: $message";
+       eval {
+           foreach my $suffix (qw(tmp git)) {
+               my $dir = "${subdir}\\.$suffix";
+               my $errs;
+               remove_tree($dir, { safe=>1, error=>\$errs });
+               if (stat $dir) {
+                   foreach my $err (@$errs) {
+                       my ($file, $message) = %$err;
+                       logm 'info', "problem deleting: $file: $message";
+                   }
+                   die "$dir: problem deleting file(s)\n";

                }
            }
                }
            }

-       }
-       if ($ok) {
+       };
+       if (length $@) {
+           chomp $@;
+           logm 'warning', "housekeeping: $subdir: cleanup prevented: $@";
+       } else {

            unlink $lock or hkfail "remove $lock: $!";
        }
     }
            unlink $lock or hkfail "remove $lock: $!";
        }
     }