PackWriter _abort: try to always close the file

...even if the unlink fails.

Signed-off-by: Rob Browning <rlb@defaultvalue.org>
Tested-by: Rob Browning <rlb@defaultvalue.org>

diff --git a/lib/bup/git.py b/lib/bup/git.py
index fd0f3c92799393b1121228a6fbd6ebed064b2c43..2f331713578a599464ed5918e4fd1a99f70026d6 100644 (file)
--- a/lib/bup/git.py
+++ b/lib/bup/git.py
@@ -685,8 +685,10 @@ class PackWriter:
         if f:
             self.idx = None
             self.file = None
-            f.close()
-            os.unlink(self.filename + '.pack')
+            try:
+                os.unlink(self.filename + '.pack')
+            finally:
+                f.close()
 
     def _end(self, run_midx=True):
         f = self.file