]> git.ipfire.org Git - thirdparty/git.git/commit
dir: fix directory-matching bug
authorDerrick Stolee <dstolee@microsoft.com>
Tue, 2 Nov 2021 14:40:06 +0000 (10:40 -0400)
committerJunio C Hamano <gitster@pobox.com>
Wed, 3 Nov 2021 17:10:36 +0000 (10:10 -0700)
commit5ceb663e926bd22248d1d72d70fa701c558587ea
tree51c8c25d8fedec667a758da720b4d710f0038aec
parent6579e788c0a4b9468c5e2954a0868f9db0496e43
dir: fix directory-matching bug

This reverts the change from ed49584 (dir: fix pattern matching on dirs,
2021-09-24), which claimed to fix a directory-matching problem without a
test case. It turns out to _create_ a bug, but it is a bit subtle.

The bug would have been revealed by the first of two tests being added to
t0008-ignores.sh. The first uses a pattern "/git/" inside the a/.gitignores
file, which matches against 'a/git/foo' but not 'a/git-foo/bar'. This test
would fail before the revert.

The second test shows what happens if the test instead uses a pattern "git/"
and this test passes both before and after the revert.

The difference in these two cases are due to how
last_matching_pattern_from_list() checks patterns both if they have the
PATTERN_FLAG_MUSTBEDIR and PATTERN_FLAG_NODIR flags. In the case of "git/",
the PATTERN_FLAG_NODIR is also provided, making the change in behavior in
match_pathname() not affect the end result of
last_matching_pattern_from_list().

Reported-by: Glen Choo <chooglen@google.com>
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
dir.c
t/t0008-ignores.sh