[PATCH] localrepo: attempt to document return values of push-related methods

Matt Mackall mpm at selenic.com
Fri Jan 15 17:21:23 CST 2010


On Thu, 2010-01-14 at 19:18 -0500, Greg Ward wrote:
>      def push(self, remote, force=False, revs=None):
> +        '''Push outgoing changesets (limited by revs) from the current
> +        repository to remote.  Return value is apparently meaningless.'''

I think it propagates the result of localrepo.addchangegroup, either
through:

push
  push_addchangegroup
    remote.addchangegroup

or

push
  push_unbundle
    httprepo.unbundle
      hgweb/protocol:unbundle
        addchangegroup

or

push
  push_unbundle
    sshrepo.unbundle
      sshserver:do_unbundle
        addchangegroup
      
And addchangegroup says:
        return values:                                                          
        - nothing changed or no source: 0                                       
        - more heads than before: 1+added heads (2..n)                          
        - less heads than before: -1-removed heads (-2..-n)                     
        - number of heads stays the same: 1          

-- 
http://selenic.com : development and support for Mercurial and Linux




More information about the Mercurial-devel mailing list