[PATCH 07 of 10] largefiles: use "summaryremotehooks" to avoid redundant outgoing check
FUJIWARA Katsunori
foozy at lares.dti.ne.jp
Wed Mar 5 06:12:12 CST 2014
# HG changeset patch
# User FUJIWARA Katsunori <foozy at lares.dti.ne.jp>
# Date 1394019743 -32400
# Wed Mar 05 20:42:23 2014 +0900
# Node ID 3fd386676d872e404dce6ca57ce739518a639bcb
# Parent 1746e2332e7188112bf4955dc8462496347fc5b2
largefiles: use "summaryremotehooks" to avoid redundant outgoing check
Before this patch, "hg summary --remote --large" invokes
"findcommonoutgoing()" not only in "commands.summary()" but also in
"overrides.overridesummary()" (via "getoutgoinglfiles()"). The latter
is redundant.
This patch uses "summaryremotehooks" to avoid redundant outgoing check.
Newly introduced function "overrides.summaryremotehook()" is
registered into "summaryremotehooks" to get the result of outgoing
check in "commands.summary()".
It invokes "lfutil.getlfilestoupload()" directly with the result of
outgoing check to avoid redundant outgoing check in
"getoutgoinglfiles()".
diff --git a/hgext/largefiles/overrides.py b/hgext/largefiles/overrides.py
--- a/hgext/largefiles/overrides.py
+++ b/hgext/largefiles/overrides.py
@@ -1018,6 +1018,27 @@
return result
+def summaryremotehook(ui, repo, opts, changes):
+ largeopt = opts.get('large', False)
+ if changes is None:
+ if largeopt:
+ return (False, True) # only outgoing check is needed
+ else:
+ return (False, False)
+ elif largeopt:
+ url, branch, peer, outgoing = changes[1]
+ if peer is None:
+ # i18n: column positioning for "hg summary"
+ ui.status(_('largefiles: (no remote repo)\n'))
+ return
+ toupload = lfutil.getlfilestoupload(repo, outgoing.missing)
+ if not toupload:
+ # i18n: column positioning for "hg summary"
+ ui.status(_('largefiles: (no files to upload)\n'))
+ else:
+ # i18n: column positioning for "hg summary"
+ ui.status(_('largefiles: %d to upload\n') % len(toupload))
+
def overridesummary(orig, ui, repo, *pats, **opts):
try:
repo.lfstatus = True
@@ -1025,18 +1046,6 @@
finally:
repo.lfstatus = False
- if opts.pop('large', None):
- toupload = getoutgoinglfiles(ui, repo, None, **opts)
- if toupload is None:
- # i18n: column positioning for "hg summary"
- ui.status(_('largefiles: (no remote repo)\n'))
- elif not toupload:
- # i18n: column positioning for "hg summary"
- ui.status(_('largefiles: (no files to upload)\n'))
- else:
- # i18n: column positioning for "hg summary"
- ui.status(_('largefiles: %d to upload\n') % len(toupload))
-
def scmutiladdremove(orig, repo, pats=[], opts={}, dry_run=None,
similarity=None):
if not lfutil.islfilesrepo(repo):
diff --git a/hgext/largefiles/uisetup.py b/hgext/largefiles/uisetup.py
--- a/hgext/largefiles/uisetup.py
+++ b/hgext/largefiles/uisetup.py
@@ -73,6 +73,7 @@
overrides.overridesummary)
summaryopt = [('', 'large', None, _('display outgoing largefiles'))]
entry[1].extend(summaryopt)
+ cmdutil.summaryremotehooks.add('largefiles', overrides.summaryremotehook)
entry = extensions.wrapcommand(commands.table, 'update',
overrides.overrideupdate)
diff --git a/tests/test-largefiles.t b/tests/test-largefiles.t
--- a/tests/test-largefiles.t
+++ b/tests/test-largefiles.t
@@ -2149,7 +2149,6 @@
branch: default
commit: (clean)
update: (current)
- searching for changes
largefiles: (no files to upload)
$ hg -R clone2 outgoing --large
comparing with $TESTTMP/issue3651/src (glob)
@@ -2172,7 +2171,6 @@
branch: default
commit: (clean)
update: (current)
- searching for changes
largefiles: 1 to upload
$ hg -R clone2 outgoing --large
comparing with $TESTTMP/issue3651/src (glob)
More information about the Mercurial-devel
mailing list