util.unlink should only catch OSError.
authorVadim Gelfer <vadim.gelfer@gmail.com>
Thu, 13 Apr 2006 13:46:56 -0700
changeset 2064 547ede0123a2
parent 2063 f1fda71e134e
child 2065 2ff37e3bf780
child 2069 15ec724ba351
util.unlink should only catch OSError.
mercurial/util.py
--- a/mercurial/util.py	Thu Apr 13 13:46:05 2006 -0700
+++ b/mercurial/util.py	Thu Apr 13 13:46:56 2006 -0700
@@ -373,8 +373,10 @@
     """unlink and remove the directory if it is empty"""
     os.unlink(f)
     # try removing directories that might now be empty
-    try: os.removedirs(os.path.dirname(f))
-    except: pass
+    try:
+        os.removedirs(os.path.dirname(f))
+    except OSError:
+        pass
 
 def copyfiles(src, dst, hardlink=None):
     """Copy a directory tree using hardlinks if possible"""