Patchwork summary: remove passing of rev because summary does not have this

login
register
mail settings
Submitter Simon Heimberg
Date July 1, 2013, 9:57 p.m.
Message ID <6cb07aa85b855f0a989c.1372715837@lapsi.heimberg.home>
Download mbox | patch
Permalink /patch/1782/
State Accepted, archived
Headers show

Comments

Simon Heimberg - July 1, 2013, 9:57 p.m.
# HG changeset patch
# User Simon Heimberg <simohe@besonet.ch>
# Date 1352157776 -3600
# Node ID 6cb07aa85b855f0a989c5d6156db6029c7016098
# Parent  660381169817a7ef8726288932cc31e98a2d94b2
summary: remove passing of rev because summary does not have this

The command summary does not have a rev argument, so passing this results
is None always. Therefore pass None explicitly.
Matt Mackall - July 2, 2013, 9:26 p.m.
On Mon, 2013-07-01 at 23:57 +0200, Simon Heimberg wrote:
> # HG changeset patch
> # User Simon Heimberg <simohe@besonet.ch>
> # Date 1352157776 -3600
> # Node ID 6cb07aa85b855f0a989c5d6156db6029c7016098
> # Parent  660381169817a7ef8726288932cc31e98a2d94b2
> summary: remove passing of rev because summary does not have this

Queued for default, thanks.

Patch

diff -r 660381169817 -r 6cb07aa85b85 mercurial/commands.py
--- a/mercurial/commands.py	Mon Jul 01 23:47:56 2013 +0200
+++ b/mercurial/commands.py	Die Nov 06 00:22:56 2012 +0100
@@ -5509,8 +5509,7 @@ 
         source, branches = hg.parseurl(ui.expandpath('default'))
         sbranch = branches[0]
         other = hg.peer(repo, {}, source)
-        revs, checkout = hg.addbranchrevs(repo, other, branches,
-                                          opts.get('rev'))
+        revs, checkout = hg.addbranchrevs(repo, other, branches, None)
         if revs:
             revs = [other.lookup(rev) for rev in revs]
         ui.debug('comparing with %s\n' % util.hidepassword(source))