threads / patch / 18133

patchClarify the "cannot lock existing info/refs" error

Subject: [PATCH] Clarify the "cannot lock existing info/refs" error

## tl;dr

4 messages between Mar 4, 2009 and Mar 5, 2009. Diffs are folded; open one to read it.

replies: 3people: 2as markdown or json

John Tapsell· Mar 4, 2009, 15:37 UTC · lore

---
 http-push.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Show changes to http-push.c +1 −1

diff --git a/http-push.c b/http-push.c
index 30d2d34..6666956 100644
--- a/http-push.c
+++ b/http-push.c
@@ -2293,7 +2293,7 @@ int main(int argc, char **argv)
 		if (info_ref_lock)
 			remote->can_update_info_refs = 1;
 		else {
-			fprintf(stderr, "Error: cannot lock existing info/refs\n");
+			error("cannot lock existing info/refs on remote server\nPerhaps the server is currently busy, or your ~/.netrc file is not configured correctly.");
 			rc = 1;
 			goto cleanup;
 		}

-- 
1.6.2.rc2.23.g77740

Clemens Buchacher· Mar 4, 2009, 19:28 UTC · re: John Tapsell · lore

Re: [PATCH] Clarify the "cannot lock existing info/refs" error

On Wed, Mar 04, 2009 at 03:37:06PM +0000, John Tapsell wrote:

> -fprintf(stderr, "Error: cannot lock existing info/refs\n");
> +error("cannot lock existing info/refs on remote server\nPerhaps the
> server is currently busy, or your ~/.netrc file is not configured
> correctly.");

In my experience this is usually caused by http-push crashing and leaving stale locks behind until it times out after 10 minutes. I don't think we should speculate here unless we can narrow down the error condition.

John Tapsell· Mar 5, 2009, 08:00 UTC · re: Clemens Buchacher · lore

Re: [PATCH] Clarify the "cannot lock existing info/refs" error

2009/3/4 Clemens Buchacher <drizzd@aon.at>:

Show 9 quoted lines

> On Wed, Mar 04, 2009 at 03:37:06PM +0000, John Tapsell wrote:
>> -fprintf(stderr, "Error: cannot lock existing info/refs\n");
>> +error("cannot lock existing info/refs on remote server\nPerhaps the
>> server is currently busy, or your ~/.netrc file is not configured
>> correctly.");
>
> In my experience this is usually caused by http-push crashing and leaving
> stale locks behind until it times out after 10 minutes. I don't think we
> should speculate here unless we can narrow down the error condition.

Yeah, I was thinking of trying to narrow it down as well.

I personally get that error 3 out of 4 times, roughly, that I try to push. But that's probably because I'm using a very busy git server. So in my case, I just keep retrying until it succeeds.

On google I found that people had been getting that error if they have the wrong password.

Can we at least change it to:

-fprintf(stderr, "Error: cannot lock existing info/refs\n"); +error("cannot lock existing info/refs on remote server\n");

It's currently confusing as to whether it's a local error or a remote error.

John

Clemens Buchacher· Mar 5, 2009, 08:50 UTC · re: John Tapsell · lore

Re: [PATCH] Clarify the "cannot lock existing info/refs" error

On Thu, Mar 05, 2009 at 08:00:37AM +0000, John Tapsell wrote:

> On google I found that people had been getting that error if they have
> the wrong password.

That's easy enough to verify. Using the wrong username/password I get

	error: Cannot access URL $url, return code 22
	error: failed to push some refs to '$url'

which is not very helpful, but otherwise unrelated.

> -fprintf(stderr, "Error: cannot lock existing info/refs\n");
> +error("cannot lock existing info/refs on remote server\n");

That's a less ambiguous but equally unhelpful error message.