Mercurial > public > mercurial-scm > hg-stable
comparison mercurial/scmutil.py @ 25122:755d23a49170
match: resolve filesets in subrepos for commands given the '-S' argument
This will work for any command that creates its matcher via scmutil.match(), but
only the files command is tested here (both workingctx and basectx based tests).
The previous behavior was to completely ignore the files in the subrepo, even
though -S was given.
My first attempt was to teach context.walk() to optionally recurse, but once
that was in place and the complete file list was built up, the predicate test
would fail with 'path in nested repo' when a file in a subrepo was accessed
through the parent context.
There are two slightly surprising behaviors with this functionality. First, any
path provided inside the fileset isn't narrowed when it is passed to the
subrepo. I dont see any clean way to do that in the matcher. Fortunately, the
'subrepo()' fileset is the only one to take a path.
The second surprise is that status predicates are resolved against the subrepo,
not the parent like 'hg status -S' is. I don't see any way to fix that either,
given the path auditor error mentioned above.
author | Matt Harbison <matt_harbison@yahoo.com> |
---|---|
date | Sat, 16 May 2015 00:36:35 -0400 |
parents | 5abd0a76bc8f |
children | 5909ac39b86a |
comparison
equal
deleted
inserted
replaced
25121:df63d4843581 | 25122:755d23a49170 |
---|---|
801 pats = [] | 801 pats = [] |
802 if not globbed and default == 'relpath': | 802 if not globbed and default == 'relpath': |
803 pats = expandpats(pats or []) | 803 pats = expandpats(pats or []) |
804 | 804 |
805 m = ctx.match(pats, opts.get('include'), opts.get('exclude'), | 805 m = ctx.match(pats, opts.get('include'), opts.get('exclude'), |
806 default) | 806 default, listsubrepos=opts.get('subrepos')) |
807 def badfn(f, msg): | 807 def badfn(f, msg): |
808 ctx.repo().ui.warn("%s: %s\n" % (m.rel(f), msg)) | 808 ctx.repo().ui.warn("%s: %s\n" % (m.rel(f), msg)) |
809 m.bad = badfn | 809 m.bad = badfn |
810 if m.always(): | 810 if m.always(): |
811 pats = [] | 811 pats = [] |