# HG changeset patch # User Martin von Zweigbergk # Date 1494998142 25200 # Node ID 12e241b2713ce44897c0bd19e8e16163b1b5d6e7 # Parent a83a7d27911ebfc0e5dff3255c215bf02b9e2064 match: remove support for excludes from matcher class The support is now provided by differencematcher() and still available via the match() function. diff -r a83a7d27911e -r 12e241b2713c mercurial/match.py --- a/mercurial/match.py Tue May 16 16:36:48 2017 -0700 +++ b/mercurial/match.py Tue May 16 22:15:42 2017 -0700 @@ -142,12 +142,12 @@ kindpats.append((kind, pats, source)) return kindpats - m = matcher(root, cwd, normalize, patterns, include=include, exclude=None, + m = matcher(root, cwd, normalize, patterns, include=include, default=default, exact=exact, auditor=auditor, ctx=ctx, listsubrepos=listsubrepos, warn=warn, badfn=badfn) if exclude: - em = matcher(root, cwd, normalize, [], include=exclude, exclude=None, - default=default, exact=False, auditor=auditor, ctx=ctx, + em = matcher(root, cwd, normalize, [], include=exclude, default=default, + exact=False, auditor=auditor, ctx=ctx, listsubrepos=listsubrepos, warn=warn, badfn=None) m = differencematcher(m, em) return m @@ -305,25 +305,21 @@ class matcher(basematcher): def __init__(self, root, cwd, normalize, patterns, include=None, - exclude=None, default='glob', exact=False, auditor=None, - ctx=None, listsubrepos=False, warn=None, badfn=None): + default='glob', exact=False, auditor=None, ctx=None, + listsubrepos=False, warn=None, badfn=None): super(matcher, self).__init__(root, cwd, badfn) if include is None: include = [] - if exclude is None: - exclude = [] - self._anypats = bool(include or exclude) + self._anypats = bool(include) self._anyincludepats = False self._always = False - self._pathrestricted = bool(include or exclude or patterns) + self._pathrestricted = bool(include or patterns) self.patternspat = None self.includepat = None - self.excludepat = None - # roots are directories which are recursively included/excluded. + # roots are directories which are recursively included. self._includeroots = set() - self._excluderoots = set() # dirs are directories which are non-recursively included. self._includedirs = set() @@ -337,18 +333,6 @@ self._includeroots.update(roots) self._includedirs.update(dirs) matchfns.append(im) - if exclude: - kindpats = normalize(exclude, 'glob', root, cwd, auditor, warn) - self.excludepat, em = _buildmatch(ctx, kindpats, '(?:/|$)', - listsubrepos, root) - if not _anypats(kindpats): - # Only consider recursive excludes as such - if a non-recursive - # exclude is used, we must still recurse into the excluded - # directory, at least to find subdirectories. In such a case, - # the regex still won't match the non-recursively-excluded - # files. - self._excluderoots.update(_roots(kindpats)) - matchfns.append(lambda f: not em(f)) if exact: if isinstance(patterns, list): self._files = patterns @@ -388,11 +372,8 @@ def visitdir(self, dir): if self.prefix() and dir in self._fileset: return 'all' - if dir in self._excluderoots: - return False if self._includeroots or self._includedirs: if (not self._anyincludepats and - not self._excluderoots and dir in self._includeroots): # The condition above is essentially self.prefix() for includes return 'all' @@ -419,9 +400,8 @@ return self.matchfn == self.exact def __repr__(self): - return ('' % - (self._files, self.patternspat, self.includepat, - self.excludepat)) + return ('' % + (self._files, self.patternspat, self.includepat)) class differencematcher(basematcher): '''Composes two matchers by matching if the first matches and the second diff -r a83a7d27911e -r 12e241b2713c tests/test-eolfilename.t --- a/tests/test-eolfilename.t Tue May 16 16:36:48 2017 -0700 +++ b/tests/test-eolfilename.t Tue May 16 22:15:42 2017 -0700 @@ -33,7 +33,7 @@ [255] $ echo foo > "$A" $ hg debugwalk - matcher: + matcher: f he\r (no-eol) (esc) llo he\r (no-eol) (esc) llo diff -r a83a7d27911e -r 12e241b2713c tests/test-hgignore.t --- a/tests/test-hgignore.t Tue May 16 16:36:48 2017 -0700 +++ b/tests/test-hgignore.t Tue May 16 22:15:42 2017 -0700 @@ -164,7 +164,7 @@ A b.o $ hg debugignore - + $ hg debugignore b.o b.o is ignored diff -r a83a7d27911e -r 12e241b2713c tests/test-walk.t --- a/tests/test-walk.t Tue May 16 16:36:48 2017 -0700 +++ b/tests/test-walk.t Tue May 16 22:15:42 2017 -0700 @@ -29,7 +29,7 @@ $ hg commit -m "commit #0" $ hg debugwalk - matcher: + matcher: f beans/black beans/black f beans/borlotti beans/borlotti f beans/kidney beans/kidney @@ -44,7 +44,7 @@ f mammals/Procyonidae/raccoon mammals/Procyonidae/raccoon f mammals/skunk mammals/skunk $ hg debugwalk -I. - matcher: + matcher: f beans/black beans/black f beans/borlotti beans/borlotti f beans/kidney beans/kidney @@ -61,7 +61,7 @@ $ cd mammals $ hg debugwalk - matcher: + matcher: f beans/black ../beans/black f beans/borlotti ../beans/borlotti f beans/kidney ../beans/kidney @@ -76,7 +76,7 @@ f mammals/Procyonidae/raccoon Procyonidae/raccoon f mammals/skunk skunk $ hg debugwalk -X ../beans - matcher: , m2=> + matcher: , m2=> f fennel ../fennel f fenugreek ../fenugreek f fiddlehead ../fiddlehead @@ -85,31 +85,31 @@ f mammals/Procyonidae/raccoon Procyonidae/raccoon f mammals/skunk skunk $ hg debugwalk -I '*k' - matcher: + matcher: f mammals/skunk skunk $ hg debugwalk -I 'glob:*k' - matcher: + matcher: f mammals/skunk skunk $ hg debugwalk -I 'relglob:*k' - matcher: + matcher: f beans/black ../beans/black f fenugreek ../fenugreek f mammals/skunk skunk $ hg debugwalk -I 'relglob:*k' . - matcher: + matcher: f mammals/skunk skunk $ hg debugwalk -I 're:.*k$' - matcher: + matcher: f beans/black ../beans/black f fenugreek ../fenugreek f mammals/skunk skunk $ hg debugwalk -I 'relre:.*k$' - matcher: + matcher: f beans/black ../beans/black f fenugreek ../fenugreek f mammals/skunk skunk $ hg debugwalk -I 'path:beans' - matcher: + matcher: f beans/black ../beans/black f beans/borlotti ../beans/borlotti f beans/kidney ../beans/kidney @@ -117,7 +117,7 @@ f beans/pinto ../beans/pinto f beans/turtle ../beans/turtle $ hg debugwalk -I 'relpath:detour/../../beans' - matcher: + matcher: f beans/black ../beans/black f beans/borlotti ../beans/borlotti f beans/kidney ../beans/kidney @@ -126,27 +126,27 @@ f beans/turtle ../beans/turtle $ hg debugwalk 'rootfilesin:' - matcher: + matcher: f fennel ../fennel f fenugreek ../fenugreek f fiddlehead ../fiddlehead $ hg debugwalk -I 'rootfilesin:' - matcher: + matcher: f fennel ../fennel f fenugreek ../fenugreek f fiddlehead ../fiddlehead $ hg debugwalk 'rootfilesin:.' - matcher: + matcher: f fennel ../fennel f fenugreek ../fenugreek f fiddlehead ../fiddlehead $ hg debugwalk -I 'rootfilesin:.' - matcher: + matcher: f fennel ../fennel f fenugreek ../fenugreek f fiddlehead ../fiddlehead $ hg debugwalk -X 'rootfilesin:' - matcher: , m2=> + matcher: , m2=> f beans/black ../beans/black f beans/borlotti ../beans/borlotti f beans/kidney ../beans/kidney @@ -158,15 +158,15 @@ f mammals/Procyonidae/raccoon Procyonidae/raccoon f mammals/skunk skunk $ hg debugwalk 'rootfilesin:fennel' - matcher: + matcher: $ hg debugwalk -I 'rootfilesin:fennel' - matcher: + matcher: $ hg debugwalk 'rootfilesin:skunk' - matcher: + matcher: $ hg debugwalk -I 'rootfilesin:skunk' - matcher: + matcher: $ hg debugwalk 'rootfilesin:beans' - matcher: + matcher: f beans/black ../beans/black f beans/borlotti ../beans/borlotti f beans/kidney ../beans/kidney @@ -174,7 +174,7 @@ f beans/pinto ../beans/pinto f beans/turtle ../beans/turtle $ hg debugwalk -I 'rootfilesin:beans' - matcher: + matcher: f beans/black ../beans/black f beans/borlotti ../beans/borlotti f beans/kidney ../beans/kidney @@ -182,19 +182,19 @@ f beans/pinto ../beans/pinto f beans/turtle ../beans/turtle $ hg debugwalk 'rootfilesin:mammals' - matcher: + matcher: f mammals/skunk skunk $ hg debugwalk -I 'rootfilesin:mammals' - matcher: + matcher: f mammals/skunk skunk $ hg debugwalk 'rootfilesin:mammals/' - matcher: + matcher: f mammals/skunk skunk $ hg debugwalk -I 'rootfilesin:mammals/' - matcher: + matcher: f mammals/skunk skunk $ hg debugwalk -X 'rootfilesin:mammals' - matcher: , m2=> + matcher: , m2=> f beans/black ../beans/black f beans/borlotti ../beans/borlotti f beans/kidney ../beans/kidney @@ -209,31 +209,31 @@ f mammals/Procyonidae/raccoon Procyonidae/raccoon $ hg debugwalk . - matcher: + matcher: f mammals/Procyonidae/cacomistle Procyonidae/cacomistle f mammals/Procyonidae/coatimundi Procyonidae/coatimundi f mammals/Procyonidae/raccoon Procyonidae/raccoon f mammals/skunk skunk $ hg debugwalk -I. - matcher: + matcher: f mammals/Procyonidae/cacomistle Procyonidae/cacomistle f mammals/Procyonidae/coatimundi Procyonidae/coatimundi f mammals/Procyonidae/raccoon Procyonidae/raccoon f mammals/skunk skunk $ hg debugwalk Procyonidae - matcher: + matcher: f mammals/Procyonidae/cacomistle Procyonidae/cacomistle f mammals/Procyonidae/coatimundi Procyonidae/coatimundi f mammals/Procyonidae/raccoon Procyonidae/raccoon $ cd Procyonidae $ hg debugwalk . - matcher: + matcher: f mammals/Procyonidae/cacomistle cacomistle f mammals/Procyonidae/coatimundi coatimundi f mammals/Procyonidae/raccoon raccoon $ hg debugwalk .. - matcher: + matcher: f mammals/Procyonidae/cacomistle cacomistle f mammals/Procyonidae/coatimundi coatimundi f mammals/Procyonidae/raccoon raccoon @@ -241,7 +241,7 @@ $ cd .. $ hg debugwalk ../beans - matcher: + matcher: f beans/black ../beans/black f beans/borlotti ../beans/borlotti f beans/kidney ../beans/kidney @@ -249,7 +249,7 @@ f beans/pinto ../beans/pinto f beans/turtle ../beans/turtle $ hg debugwalk . - matcher: + matcher: f mammals/Procyonidae/cacomistle Procyonidae/cacomistle f mammals/Procyonidae/coatimundi Procyonidae/coatimundi f mammals/Procyonidae/raccoon Procyonidae/raccoon @@ -263,7 +263,7 @@ $ cd .. $ hg debugwalk -Ibeans - matcher: + matcher: f beans/black beans/black f beans/borlotti beans/borlotti f beans/kidney beans/kidney @@ -271,56 +271,56 @@ f beans/pinto beans/pinto f beans/turtle beans/turtle $ hg debugwalk -I '{*,{b,m}*/*}k' - matcher: + matcher: f beans/black beans/black f fenugreek fenugreek f mammals/skunk mammals/skunk $ hg debugwalk -Ibeans mammals - matcher: + matcher: $ hg debugwalk -Inon-existent - matcher: + matcher: $ hg debugwalk -Inon-existent -Ibeans/black - matcher: + matcher: f beans/black beans/black $ hg debugwalk -Ibeans beans/black - matcher: + matcher: f beans/black beans/black exact $ hg debugwalk -Ibeans/black beans - matcher: + matcher: f beans/black beans/black $ hg debugwalk -Xbeans/black beans - matcher: , m2=> + matcher: , m2=> f beans/borlotti beans/borlotti f beans/kidney beans/kidney f beans/navy beans/navy f beans/pinto beans/pinto f beans/turtle beans/turtle $ hg debugwalk -Xbeans/black -Ibeans - matcher: , m2=> + matcher: , m2=> f beans/borlotti beans/borlotti f beans/kidney beans/kidney f beans/navy beans/navy f beans/pinto beans/pinto f beans/turtle beans/turtle $ hg debugwalk -Xbeans/black beans/black - matcher: , m2=> + matcher: , m2=> f beans/black beans/black exact $ hg debugwalk -Xbeans/black -Ibeans/black - matcher: , m2=> + matcher: , m2=> $ hg debugwalk -Xbeans beans/black - matcher: , m2=> + matcher: , m2=> f beans/black beans/black exact $ hg debugwalk -Xbeans -Ibeans/black - matcher: , m2=> + matcher: , m2=> $ hg debugwalk 'glob:mammals/../beans/b*' - matcher: + matcher: f beans/black beans/black f beans/borlotti beans/borlotti $ hg debugwalk '-X*/Procyonidae' mammals - matcher: , m2=> + matcher: , m2=> f mammals/skunk mammals/skunk $ hg debugwalk path:mammals - matcher: + matcher: f mammals/Procyonidae/cacomistle mammals/Procyonidae/cacomistle f mammals/Procyonidae/coatimundi mammals/Procyonidae/coatimundi f mammals/Procyonidae/raccoon mammals/Procyonidae/raccoon @@ -347,7 +347,7 @@ Test absolute paths: $ hg debugwalk `pwd`/beans - matcher: + matcher: f beans/black beans/black f beans/borlotti beans/borlotti f beans/kidney beans/kidney @@ -361,7 +361,7 @@ Test patterns: $ hg debugwalk glob:\* - matcher: + matcher: f fennel fennel f fenugreek fenugreek f fiddlehead fiddlehead @@ -371,19 +371,19 @@ adding glob:glob warning: filename contains ':', which is reserved on Windows: 'glob:glob' $ hg debugwalk glob:\* - matcher: + matcher: f fennel fennel f fenugreek fenugreek f fiddlehead fiddlehead f glob:glob glob:glob $ hg debugwalk glob:glob - matcher: + matcher: glob: No such file or directory $ hg debugwalk glob:glob:glob - matcher: + matcher: f glob:glob glob:glob exact $ hg debugwalk path:glob:glob - matcher: + matcher: f glob:glob glob:glob exact $ rm glob:glob $ hg addremove @@ -391,38 +391,38 @@ #endif $ hg debugwalk 'glob:**e' - matcher: + matcher: f beans/turtle beans/turtle f mammals/Procyonidae/cacomistle mammals/Procyonidae/cacomistle $ hg debugwalk 're:.*[kb]$' - matcher: + matcher: f beans/black beans/black f fenugreek fenugreek f mammals/skunk mammals/skunk $ hg debugwalk path:beans/black - matcher: + matcher: f beans/black beans/black exact $ hg debugwalk path:beans//black - matcher: + matcher: f beans/black beans/black exact $ hg debugwalk relglob:Procyonidae - matcher: + matcher: $ hg debugwalk 'relglob:Procyonidae/**' - matcher: + matcher: f mammals/Procyonidae/cacomistle mammals/Procyonidae/cacomistle f mammals/Procyonidae/coatimundi mammals/Procyonidae/coatimundi f mammals/Procyonidae/raccoon mammals/Procyonidae/raccoon $ hg debugwalk 'relglob:Procyonidae/**' fennel - matcher: + matcher: f fennel fennel exact f mammals/Procyonidae/cacomistle mammals/Procyonidae/cacomistle f mammals/Procyonidae/coatimundi mammals/Procyonidae/coatimundi f mammals/Procyonidae/raccoon mammals/Procyonidae/raccoon $ hg debugwalk beans 'glob:beans/*' - matcher: + matcher: f beans/black beans/black f beans/borlotti beans/borlotti f beans/kidney beans/kidney @@ -430,78 +430,78 @@ f beans/pinto beans/pinto f beans/turtle beans/turtle $ hg debugwalk 'glob:mamm**' - matcher: + matcher: f mammals/Procyonidae/cacomistle mammals/Procyonidae/cacomistle f mammals/Procyonidae/coatimundi mammals/Procyonidae/coatimundi f mammals/Procyonidae/raccoon mammals/Procyonidae/raccoon f mammals/skunk mammals/skunk $ hg debugwalk 'glob:mamm**' fennel - matcher: + matcher: f fennel fennel exact f mammals/Procyonidae/cacomistle mammals/Procyonidae/cacomistle f mammals/Procyonidae/coatimundi mammals/Procyonidae/coatimundi f mammals/Procyonidae/raccoon mammals/Procyonidae/raccoon f mammals/skunk mammals/skunk $ hg debugwalk 'glob:j*' - matcher: + matcher: $ hg debugwalk NOEXIST - matcher: + matcher: NOEXIST: * (glob) #if fifo $ mkfifo fifo $ hg debugwalk fifo - matcher: + matcher: fifo: unsupported file type (type is fifo) #endif $ rm fenugreek $ hg debugwalk fenugreek - matcher: + matcher: f fenugreek fenugreek exact $ hg rm fenugreek $ hg debugwalk fenugreek - matcher: + matcher: f fenugreek fenugreek exact $ touch new $ hg debugwalk new - matcher: + matcher: f new new exact $ mkdir ignored $ touch ignored/file $ echo '^ignored$' > .hgignore $ hg debugwalk ignored - matcher: + matcher: $ hg debugwalk ignored/file - matcher: + matcher: f ignored/file ignored/file exact Test listfile and listfile0 $ $PYTHON -c "file('listfile0', 'wb').write('fenugreek\0new\0')" $ hg debugwalk -I 'listfile0:listfile0' - matcher: + matcher: f fenugreek fenugreek f new new $ $PYTHON -c "file('listfile', 'wb').write('fenugreek\nnew\r\nmammals/skunk\n')" $ hg debugwalk -I 'listfile:listfile' - matcher: + matcher: f fenugreek fenugreek f mammals/skunk mammals/skunk f new new $ cd .. $ hg debugwalk -R t t/mammals/skunk - matcher: + matcher: f mammals/skunk t/mammals/skunk exact $ mkdir t2 $ cd t2 $ hg debugwalk -R ../t ../t/mammals/skunk - matcher: + matcher: f mammals/skunk ../t/mammals/skunk exact $ hg debugwalk --cwd ../t mammals/skunk - matcher: + matcher: f mammals/skunk mammals/skunk exact $ cd ..