git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] Make the 'lock file exists' error more informative

From
John Tapsell <johnflux@gmail.com>
Date
Mar 4, 2009, 15:07 UTC
Message-ID
<1236179277-12477-1-git-send-email-johnflux@gmail.com>
It looks like someone did 90% of the work, then forgot to actually use
the function
---
 lockfile.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lockfile.c b/lockfile.c
index 1db1a2f..3dbb2d1 100644
--- a/lockfile.c
+++ b/lockfile.c
@@ -158,7 +158,7 @@ static int lock_file(struct lock_file *lk, const char *path, int flags)
 
 NORETURN void unable_to_lock_index_die(const char *path, int err)
 {
-	if (errno == EEXIST) {
+	if (err == EEXIST) {
 		die("Unable to create '%s.lock': %s.\n\n"
 		    "If no other git process is currently running, this probably means a\n"
 		    "git process crashed in this repository earlier. Make sure no other git\n"
@@ -184,7 +184,7 @@ int hold_lock_file_for_append(struct lock_file *lk, const char *path, int flags)
 	fd = lock_file(lk, path, flags);
 	if (fd < 0) {
 		if (flags & LOCK_DIE_ON_ERROR)
-			die("unable to create '%s.lock': %s", path, strerror(errno));
+			unable_to_lock_index_die(path, errno);
 		return fd;
 	}
 
-- 
1.6.2.rc2.23.g77740
Next: John Tapsell
Message 1 of 4 in “Make the 'lock file exists' error more informative”
  1. Make the 'lock file exists' error more informativeJohn Tapsell, Mar 4, 2009
  2. John TapsellMar 4, 2009
  3. Matthieu MoyMar 4, 2009
  4. Junio C HamanoMar 4, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.