Mercurial > hg > mercurial-source
changeset 23753:f2893cd8d1e5
largefiles: pass a matcher instead of a raw file list to removelargefiles()
This is consistent with addlargefiles(), and will make it easier to get the
paths that are printed correct when recursing into subrepos or invoking from
outside the repository. It also now restricts the path that the addremove is
performed on if a path is given, as is done with normal files.
The repo.status() call needs to exclude clean files when performing an
addremove, because the addremove override method calling this used to pass the
list of files to delete, which caused the matcher to only consider those files
in building the status list. Now the matcher is restricted only to the extent
that the caller requested- usually directories if at all. There's no reason for
addremove to care about clean files anyway- we don't want them deleted.
author | Matt Harbison <matt_harbison@yahoo.com> |
---|---|
date | Fri, 28 Nov 2014 19:50:52 -0500 |
parents | 9e1f4c65f5f5 |
children | 3a4d8a6ce432 |
files | hgext/largefiles/overrides.py tests/test-largefiles-misc.t |
diffstat | 2 files changed, 19 insertions(+), 9 deletions(-) [+] |
line wrap: on
line diff
--- a/hgext/largefiles/overrides.py +++ b/hgext/largefiles/overrides.py @@ -152,13 +152,12 @@ wlock.release() return bad -def removelargefiles(ui, repo, isaddremove, *pats, **opts): +def removelargefiles(ui, repo, isaddremove, matcher, **opts): after = opts.get('after') - m = composelargefilematcher(scmutil.match(repo[None], pats, opts), - repo[None].manifest()) + m = composelargefilematcher(matcher, repo[None].manifest()) try: repo.lfstatus = True - s = repo.status(match=m, clean=True) + s = repo.status(match=m, clean=not isaddremove) finally: repo.lfstatus = False manifest = repo[None].manifest() @@ -250,7 +249,8 @@ installnormalfilesmatchfn(repo[None].manifest()) result = orig(ui, repo, *pats, **opts) restorematchfn() - return removelargefiles(ui, repo, False, *pats, **opts) or result + matcher = scmutil.match(repo[None], pats, opts) + return removelargefiles(ui, repo, False, matcher, **opts) or result def overridestatusfn(orig, repo, rev2, **opts): try: @@ -1109,8 +1109,16 @@ # we don't remove the standin in the largefiles code, preventing a very # confused state later. if s.deleted: - m = [repo.wjoin(f) for f in s.deleted] - removelargefiles(repo.ui, repo, True, *m, **opts) + m = copy.copy(matcher) + + # The m._files and m._map attributes are not changed to the deleted list + # because that affects the m.exact() test, which in turn governs whether + # or not the file name is printed, and how. Simply limit the original + # matches to those in the deleted status list. + matchfn = m.matchfn + m.matchfn = lambda f: f in s.deleted and matchfn(f) + + removelargefiles(repo.ui, repo, True, m, **opts) # Call into the normal add code, and any files that *should* be added as # largefiles will be addlargefiles(repo.ui, repo, matcher, **opts)
--- a/tests/test-largefiles-misc.t +++ b/tests/test-largefiles-misc.t @@ -269,11 +269,13 @@ $ mv subrepo/renamed-large.txt subrepo/large.txt $ hg -R subrepo add subrepo/normal.txt - $ hg addremove + $ hg addremove subrepo + $ hg addremove -S adding large.dat as a largefile $ rm large.dat - $ hg addremove + $ hg addremove subrepo + $ hg addremove -S removing large.dat Lock in subrepo, otherwise the change isn't archived