]> git.ipfire.org Git - thirdparty/git.git/commit - config.c
config: correct '**' matching in includeIf patterns
authorNguyễn Thái Ngọc Duy <pclouds@gmail.com>
Tue, 26 Mar 2019 09:41:01 +0000 (16:41 +0700)
committerJunio C Hamano <gitster@pobox.com>
Mon, 1 Apr 2019 05:19:47 +0000 (14:19 +0900)
commit19e7fdaa582598fb915e0a421a14b559c06587fd
treed7ad476e8dcf548f1797566529ce46f0aa8c6b59
parentaeb582a98374c094361cba1bd756dc6307432c42
config: correct '**' matching in includeIf patterns

The current wildmatch() call for includeIf's gitdir pattern does not
pass the WM_PATHNAME flag. Without this flag, '*' is treated _almost_
the same as '**' (because '*' also matches slashes) with one exception:

'/**/' can match a single slash. The pattern 'foo/**/bar' matches
'foo/bar'.

But '/*/', which is essentially what wildmatch engine sees without
WM_PATHNAME, has to match two slashes (and '*' matches nothing). Which
means 'foo/*/bar' cannot match 'foo/bar'. It can only match 'foo//bar'.

The result of this is the current wildmatch() call works most of the
time until the user depends on '/**/' matching no path component. And
also '*' matches slashes while it should not, but people probably
haven't noticed this yet. The fix is straightforward.

Reported-by: Jason Karns <jason.karns@gmail.com>
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
config.c
t/t1305-config-include.sh