diff options
author | Martin Ågren <martin.agren@gmail.com> | 2020-07-20 20:45:29 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2020-07-20 13:25:07 -0700 |
commit | cada7308ade0b9a018f9c54ea3f226e2e9c39ee5 (patch) | |
tree | 2f8e26c08a19b515d3adbacd3f81a4024e1c0da7 /Documentation/RelNotes/1.7.10.4.txt | |
parent | completion: fix 'git add' on paths under an untracked directory (diff) | |
download | tgif-cada7308ade0b9a018f9c54ea3f226e2e9c39ee5.tar.xz |
dir: check pathspecs before returning `path_excluded`
In 95c11ecc73 ("Fix error-prone fill_directory() API; make it only
return matches", 2020-04-01), we taught `fill_directory()`, or more
specifically `treat_path()`, to check against any pathspecs so that we
could simplify the callers.
But in doing so, we added a slightly-too-early return for the "excluded"
case. We end up not checking the pathspecs, meaning we return
`path_excluded` when maybe we should return `path_none`. As a result,
`git status --ignored -- pathspec` might show paths that don't actually
match "pathspec".
Move the "excluded" check down to after we've checked any pathspecs.
Reported-by: Andreas Schwab <schwab@linux-m68k.org>
Reviewed-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Martin Ågren <martin.agren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'Documentation/RelNotes/1.7.10.4.txt')
0 files changed, 0 insertions, 0 deletions