# HG changeset patch # User Pierre-Yves David # Date 1412352467 18000 # Node ID 68439b1540633fb32c67b351cc2afade22773544 # Parent 78c916f24dd99a56e4c29153a5df3bd7d1c40edd exchange: have `pull` return the pulloperation object We mimic what was done for `push` for similar reason. We are about to drop `localrepo.pull` (for consistency with dropping `localrepo.push` and we better have an API as extensible as `push` is. Find explanations about localrepo.push removal in 4d52e6eb98ea. diff -r 78c916f24dd9 -r 68439b154063 mercurial/exchange.py --- a/mercurial/exchange.py Wed Oct 01 15:14:36 2014 -0500 +++ b/mercurial/exchange.py Fri Oct 03 11:07:47 2014 -0500 @@ -866,7 +866,7 @@ pullop.releasetransaction() lock.release() - return pullop.cgresult + return pullop def _pulldiscovery(pullop): """discovery phase for the pull diff -r 78c916f24dd9 -r 68439b154063 mercurial/localrepo.py --- a/mercurial/localrepo.py Wed Oct 01 15:14:36 2014 -0500 +++ b/mercurial/localrepo.py Fri Oct 03 11:07:47 2014 -0500 @@ -1575,7 +1575,7 @@ return r def pull(self, remote, heads=None, force=False, **kwargs): - return exchange.pull(self, remote, heads, force, **kwargs) + return exchange.pull(self, remote, heads, force, **kwargs).cgresult def checkpush(self, pushop): """Extensions can override this function if additional checks have