[PATCH v2 1/1] git mv foo FOO ; git mv foo bar gave an assert

From: tboegi@web.de <tboegi@web.de>
Date: Mar 1, 2021, 17:05 UTC
Message-ID: <20210301170536.12265-1-tboegi@web.de>
In-Reply-To: <xmqq7doqwvzx.fsf@gitster.c.googlers.com>

From: Torsten Bögershausen <tboegi@web.de>

The following sequence, on a case-insensitive file system, (strictly speeking with core.ignorecase=true) leads to an assertion, and leaves .git/index.lock behind.

git init echo foo >foo git add foo git mv foo FOO git mv foo bar

This regression was introduced in Commit 9b906af657, "git-mv: improve error message for conflicted file"

The bugfix is to change the "file exist case-insensitive in the index" into the correct "file exist (case-sensitive) in the index". This avoids the "assert".

Reported-By: Dan Moseley <Dan.Moseley@microsoft.com>

This fixes https://github.com/git-for-windows/git/issues/2920

Signed-off-by: Torsten Bögershausen <tboegi@web.de>
---
 builtin/mv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/builtin/mv.c b/builtin/mv.c
index 7dac714af9..3fccdcb645 100644
--- a/builtin/mv.c
+++ b/builtin/mv.c
@@ -221,7 +221,7 @@ int cmd_mv(int argc, const char **argv, const char *prefix)
 				}
 				argc += last - first;
 			}
-		} else if (!(ce = cache_file_exists(src, length, ignore_case))) {
+		} else if (!(ce = cache_file_exists(src, length, 0))) {
 			bad = _("not under version control");
 		} else if (ce_stage(ce)) {
 			bad = _("conflicted");
--
2.30.0.155.g66e871b664