D1957: streamclonebundle: make sure we accept new stream clone bundle spec

indygreg (Gregory Szorc) phabricator at mercurial-scm.org
Tue Feb 20 23:53:46 EST 2018


indygreg added a comment.


  In general, I'm pretty happy with this series. Although I held off accepting changesets because I think we should have a mini discussion about //content options// versus //capabilities// before things get out of control and we duplicate too much logic.
  
  I'd also like to note that bundle specs are intended to be user facing and their behavior should be as simple as possible for people. e.g. consider adding keywords to parameters instead of requiring parameters have values.
  
  Also, while you are here, we should consider defining a `v3` bundlespec that allows embedding of bookmarks, phases, obsolescence markers, etc. Basically all the modern features of bundle2 that modern peers and servers will negotiate with each other.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D1957

To: lothiraldan, #hg-reviewers
Cc: indygreg, mercurial-devel


More information about the Mercurial-devel mailing list