[PATCH 9 of 9] subrepo: drop the 'ui' parameter to revert()

Matt Mackall mpm at selenic.com
Thu Dec 18 13:07:06 CST 2014


On Thu, 2014-12-18 at 13:17 -0500, Matt Harbison wrote:
> On Dec 18, 2014 1:01 PM, Matt Mackall <mpm at selenic.com> wrote: 
> 
> On Sun, 2014-12-14 at 20:12 -0500, Matt Harbison wrote: 
> > # HG changeset patch 
> > # User Matt Harbison <matt_harbison at yahoo.com> 
> > # Date 1418517895 18000 
> > # Sat Dec 13 19:44:55 2014 -0500 
> > # Node ID cd72135f7096f311371a86fab038d0a27c3fea58 
> > # Parent b58f5dcfe069ccf5154788c05348cc328b4f13f2 
> > subrepo: drop the 'ui' parameter to revert() 
> 
> FYI, this broke test-subrepo-git.t. I've fixed it in flight. 
> 
> I think it ran fine for me, minus the recent couple of whitespace
> changes dependent on the git version.  Patches for that were accepted
> over the weekend.  I'm a little concerned if refactoring changed
> tests.

This bit was missing in the version you posted. Not sure how it got past
Augie, who presumably has git installed.

diff -r e1c39f207719 -r e20f36ad092e mercurial/subrepo.py
--- a/mercurial/subrepo.py	Sat Dec 13 19:44:55 2014 -0500
+++ b/mercurial/subrepo.py	Thu Dec 18 12:07:03 2014 -0600
@@ -1655,12 +1655,12 @@
         elif match(gitprefix): #Subrepo is matched
             ui.write(self._gitcommand(cmd))
 
-    def revert(self, ui, substate, *pats, **opts):
-        ui.status(_('reverting subrepo %s\n') % substate[0])
+    def revert(self, substate, *pats, **opts):
+        self.ui.status(_('reverting subrepo %s\n') % substate[0])
         if not opts.get('no_backup'):
-            ui.warn('%s: reverting %s subrepos without '
-                    '--no-backup is unsupported\n'
-                    % (substate[0], substate[2]))
+            self.ui.warn('%s: reverting %s subrepos without '
+                         '--no-backup is unsupported\n'
+                         % (substate[0], substate[2]))
             return []
 
         self.get(substate, overwrite=True)


-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list