]> git.ipfire.org Git - thirdparty/git.git/commit - path.c
test-path-utils.c: remove incorrect assumption
authorRay Donnelly <mingw.android@gmail.com>
Thu, 1 Oct 2015 19:04:17 +0000 (20:04 +0100)
committerJunio C Hamano <gitster@pobox.com>
Fri, 9 Oct 2015 01:03:50 +0000 (18:03 -0700)
commitb2a7123b997f950e9785a5e7df64c3104270fef3
treee6536372e9574e61c1d56d12263b6812389bf08d
parent22f698cb188243b313e024d618283e0293e37140
test-path-utils.c: remove incorrect assumption

In normalize_ceiling_entry(), we test that normalized paths end with
slash, *unless* the path to be normalized was already the root
directory.

However, normalize_path_copy() does not even enforce this condition.

Even worse: on Windows, the root directory gets translated into a
Windows directory by the Bash before being passed to `git.exe` (or
`test-path-utils.exe`), which means that we cannot even know whether
the path that was passed to us was the root directory to begin with.

This issue has already caused endless hours of trying to "fix" the
MSYS2 runtime, only to break other things due to MSYS2 ensuring that
the converted path maintains the same state as the input path with
respect to any final '/'.

So let's just forget about this test. It is non-essential to Git's
operation, anyway.

Acked-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Ray Donnelly <mingw.android@gmail.com>
path.c
test-path-utils.c