D1063: rebase: enable multidest by default

quark (Jun Wu) phabricator at mercurial-scm.org
Tue Oct 17 13:54:01 EDT 2017


quark added a comment.


  https://phab.mercurial-scm.org/D1139 changes restack to use revsets. The resulting code is simpler, and no longer do duplicated rebases.
  
  From a high level, this feature basically allows some user-invisible logic like rebase source and destination decision to be more user-visible as revsets, so users can investigate them by running `log -r` etc., which seems to be a good thing.
  
  > Ultimately, if it's really safe, why keeping the option at all?
  
  The feature was implemented by multiple patches. I added the config so in case some of the patches did not land, the incomplete feature is not exposed. Now that all patches are landed. It's time to enable the feature by default.

INLINE COMMENTS

> dlax wrote in rebase.py:698
> I'd also be very useful to have an example of how to stabilize only the current stack, but maybe this is unrelated to this changeset.

This could be expressed via a complex revset if `predecessors` is in core.

REPOSITORY
  rHG Mercurial

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

To: quark, #hg-reviewers, durin42
Cc: dlax, martinvonz, durin42, mercurial-devel


More information about the Mercurial-devel mailing list