indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers.
REVISION SUMMARY The thing that matters is that self._clnodetorev is populated with changesets that are being sent. Back when this code was in an extension, it wasn't possible to monkeypatch the changelog lookup function. Now that the code is in core, we can move this code to where it logically belongs. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D4186 AFFECTED FILES mercurial/changegroup.py CHANGE DETAILS diff --git a/mercurial/changegroup.py b/mercurial/changegroup.py --- a/mercurial/changegroup.py +++ b/mercurial/changegroup.py @@ -852,6 +852,8 @@ clrevorder[x] = len(clrevorder) if self._ellipses: + self._clnodetorev[x] = cl.rev(x) + # Only update mfs if x is going to be sent. Otherwise we # end up with bogus linkrevs specified for manifests and # we skip some manifest nodes that we should otherwise @@ -1045,15 +1047,8 @@ progress.complete() def _revisiondeltanarrow(self, store, ischangelog, rev, prev, linknode): - # build up some mapping information that's useful later. See - # the local() nested function below. - if ischangelog: - self._clnodetorev[linknode] = rev - linkrev = rev - self._clrevtolocalrev[linkrev] = rev - else: - linkrev = self._clnodetorev[linknode] - self._clrevtolocalrev[linkrev] = rev + linkrev = self._clnodetorev[linknode] + self._clrevtolocalrev[linkrev] = rev # This is a node to send in full, because the changeset it # corresponds to was a full changeset. To: indygreg, #hg-reviewers Cc: mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
