LockFile: release_lock now checks whether the lockfile to be removed still exists. Previously it would just fail

Byron · Byron · commit 76fd1d411924 · 2010-02-18T14:33:51.000+01:00
diff --git a/lib/git/utils.py b/lib/git/utils.py
@@ -166,7 +166,12 @@ def _release_lock(self):
 		"""
 		if not self._has_lock():
 			return
-		os.remove(self._lock_file_path())
+			
+		# if someone removed our file beforhand, lets just flag this issue
+		# instead of failing, to make it more usable.
+		lfp = self._lock_file_path()
+		if os.path.isfile(lfp):
+			os.remove(lfp)
 		self._owns_lock = False
 
 
