martinvonz added inline comments. INLINE COMMENTS
> narrowcommands.py:149 > # we have all the nodes > if wireprototypes.ELLIPSESCAP in pullop.remote.capabilities(): > kwargs['known'] = [node.hex(ctx.node()) for ctx in Also need to check old capability here > narrowcommands.py:257-258 > + remotecap = remote.capabilities() > + ellipsesremote = any(True for cap in wireprototypes.SUPPORTED_ELLIPSESCAP > + if cap in remotecap) > + nit: `any(cap in remotecap for cap in wireprototypes.SUPPORTED_ELLIPSESCAP)` > narrowcommands.py:282 > with wrappedextraprepare: > - with repo.ui.configoverride(overrides, 'widen'): > exchange.pull(repo, remote, heads=common) Should keep this in the "old" case > narrowcommands.py:303-305 > + 'commonheads': common, > + 'known': known, > + 'ellipses': True, Is the `known` set needed when not using ellipses? Conversely, `commonheads` shouldn't be needed when using ellipses, but perhaps it's still useful to have it there (it's usually way smaller, so it's much less of a concern). These things can be fixed in a separate patch, of course. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D6436 To: pulkit, durin42, martinvonz, #hg-reviewers Cc: mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
