# Bug? - Unexpected lock file error 7 messages from 2008-09-05 to 2008-09-07. Participants: Eric Raible, Junio C Hamano, Miklos Vajna. Thread: https://gitlist.dev/t/15390 ## Eric Raible, 2008-09-05 23:22 Subject: Bug? - Unexpected lock file error Message-ID: <279b37b20809051622q6b3ab37tec3f1726c71952e4@mail.gmail.com> URL: https://gitlist.dev/e/279b37b20809051622q6b3ab37tec3f1726c71952e4%40mail.gmail.com ``` In playing around with creating new unatached branches I came across a seemingly bogus "fatal: unable to create '.git/index.lock': File exists" error message. Now I understand that the final merge below is a no-op at best but the resulting error message is less than intuitive. This is in "git version 1.6.0.1.319.g9f32b.dirty" (msysgit/devel, but I doubt that makes a difference). Comments? # Make a couple of versions of 'a' in main repo mkdir foo cd foo git init -q touch a for i in a b c; do echo $i > a && git add a && git commit -q -m $i; done # Make a new repo and push it into main repo (to make an unattached branch) mkdir empty cd empty git init -q git commit -q --allow-empty -m empty git push .. master:empty # Trigger "fatal: unable to create '.git/index.lock': File exists" error cd .. git merge empty If 'empty' has any commits then the merge happens as expected. Is this just another special case involving initial commits that I should just get over? Or is there something more substantive going on here? - Eric ``` ## Junio C Hamano, 2008-09-05 23:47 Subject: Re: Bug? - Unexpected lock file error Message-ID: <7v63paupd8.fsf@gitster.siamese.dyndns.org> URL: https://gitlist.dev/e/7v63paupd8.fsf%40gitster.siamese.dyndns.org In-Reply-To: <279b37b20809051622q6b3ab37tec3f1726c71952e4@mail.gmail.com> ``` "Eric Raible" writes: > If 'empty' has any commits then the merge happens as expected. I think it is a bug in the git-merge rewritten in C, in "Already uptodate!" codepath. ``` ## Eric Raible, 2008-09-06 00:05 Subject: Re: Bug? - Unexpected lock file error Message-ID: <279b37b20809051705o1f00ac4fpfc085fe58fdf5133@mail.gmail.com> URL: https://gitlist.dev/e/279b37b20809051705o1f00ac4fpfc085fe58fdf5133%40mail.gmail.com In-Reply-To: <7v63paupd8.fsf@gitster.siamese.dyndns.org> ``` On Fri, Sep 5, 2008 at 4:47 PM, Junio C Hamano wrote: > "Eric Raible" writes: > >> If 'empty' has any commits then the merge happens as expected. > > I think it is a bug in the git-merge rewritten in C, in "Already > uptodate!" codepath. And the ""Already uptodate!" codepath isn't even triggered when we're up-to-date but 'empty' has at even one actual commit. In that case we get a friendly "Already up-to-date" and no fatal error. ``` ## Junio C Hamano, 2008-09-06 00:09 Subject: Re: Bug? - Unexpected lock file error Message-ID: <7vtzcut9s3.fsf@gitster.siamese.dyndns.org> URL: https://gitlist.dev/e/7vtzcut9s3.fsf%40gitster.siamese.dyndns.org In-Reply-To: <7v63paupd8.fsf@gitster.siamese.dyndns.org> ``` Junio C Hamano writes: > "Eric Raible" writes: > >> If 'empty' has any commits then the merge happens as expected. > > I think it is a bug in the git-merge rewritten in C, in "Already > uptodate!" codepath. Try reverting 18668f5 (builtin-merge: avoid run_command_v_opt() for recursive and subtree, 2008-08-28). That patch seems to be seriously broken. ``` ## Miklos Vajna, 2008-09-06 02:12 Subject: Re: Bug? - Unexpected lock file error Message-ID: <20080906021222.GY16514@genesis.frugalware.org> URL: https://gitlist.dev/e/20080906021222.GY16514%40genesis.frugalware.org In-Reply-To: <7vtzcut9s3.fsf@gitster.siamese.dyndns.org> ``` On Fri, Sep 05, 2008 at 05:09:32PM -0700, Junio C Hamano wrote: > > I think it is a bug in the git-merge rewritten in C, in "Already > > uptodate!" codepath. I can't reproduce it with master, so I don't think this problem was introduced by the C rewrite. > Try reverting 18668f5 (builtin-merge: avoid run_command_v_opt() for > recursive and subtree, 2008-08-28). That patch seems to be seriously > broken. Yes, that works around the problem. I'll try to send a fix later today. ``` ## Miklos Vajna, 2008-09-06 16:29 Subject: [PATCH] builtin-merge: release the lockfile in try_merge_strategy() Message-ID: <1220718589-12424-1-git-send-email-vmiklos@frugalware.org> URL: https://gitlist.dev/e/1220718589-12424-1-git-send-email-vmiklos%40frugalware.org In-Reply-To: <20080906021222.GY16514@genesis.frugalware.org> ``` Once we committed the locked index, we should release the lockfile. In most cases this is done automatically when the process ends, but this is not true in this case. Signed-off-by: Miklos Vajna --- On Sat, Sep 06, 2008 at 04:12:23AM +0200, Miklos Vajna wrote: > > Try reverting 18668f5 (builtin-merge: avoid run_command_v_opt() for > > recursive and subtree, 2008-08-28). That patch seems to be > > seriously broken. > > Yes, that works around the problem. I'll try to send a fix later > today. Here is - I think - the proper prefix. builtin-merge.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/builtin-merge.c b/builtin-merge.c index b857cf6..bb09e6f 100644 --- a/builtin-merge.c +++ b/builtin-merge.c @@ -576,6 +576,7 @@ static int try_merge_strategy(const char *strategy, struct commit_list *common, (write_cache(index_fd, active_cache, active_nr) || commit_locked_index(lock))) die ("unable to write %s", get_index_file()); + rollback_lock_file(lock); return clean ? 0 : 1; } else { args = xmalloc((4 + commit_list_count(common) + -- 1.6.0.1 ``` ## Junio C Hamano, 2008-09-07 05:44 Subject: Re: [PATCH] builtin-merge: release the lockfile in try_merge_strategy() Message-ID: <7vod30pl1o.fsf@gitster.siamese.dyndns.org> URL: https://gitlist.dev/e/7vod30pl1o.fsf%40gitster.siamese.dyndns.org In-Reply-To: <1220718589-12424-1-git-send-email-vmiklos@frugalware.org> ``` Miklos Vajna writes: > Once we committed the locked index, we should release the lockfile. In > most cases this is done automatically when the process ends, but this is > not true in this case. Thanks. ```