Page MenuHomePhabricator

Use a "branchmap" call to identify remote branches in "arc-hg"
ClosedPublic

Authored by epriestley on Jun 10 2020, 10:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 7:33 AM
Unknown Object (File)
Sat, Dec 7, 4:27 PM
Unknown Object (File)
Sat, Dec 7, 12:27 PM
Unknown Object (File)
Fri, Dec 6, 6:15 AM
Unknown Object (File)
Nov 7 2024, 9:52 AM
Unknown Object (File)
Nov 5 2024, 1:58 AM
Unknown Object (File)
Nov 3 2024, 12:00 AM
Unknown Object (File)
Oct 31 2024, 8:35 AM
Subscribers
None

Details

Summary

Ref T9948. Ref T13546. To identify remote branch heads -- not just modified heads -- use "branchmap" like "hg outgoing" does.

I wasn't able to find any other way to get what we want: for example, with a bundlerepo, "peer.heads()" and "peer.changelog.heads()" include local branches which are not present in the remote.

It generally seems difficult (perhaps impossible?) to distinguish between these cases by using "getremotechanges()":

  • branch X exists at position Y in both the local and remote;
  • branch X exists at positino Y in the local, but not the remote.

In any case, this seems to work properly and should do less work than "getremotechanges()" since we don't need to pull as much data over the wire.

Test Plan

Ran "hg arc-ls-markers" in various repositories, got what appeared to be a faithful representation of the remote branch and bookmark state.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Jun 11 2020, 12:31 AM
This revision was automatically updated to reflect the committed changes.