D6686: unshelve: handle stripping changesets on interactive mode

pulkit (Pulkit Goyal) phabricator at mercurial-scm.org
Fri Jul 26 10:44:20 EDT 2019


pulkit added a comment.


  In D6686#97957 <https://phab.mercurial-scm.org/D6686#97957>, @navaneeth.suresh wrote:
  
  > I just found that this change is not required while creating D6694 <https://phab.mercurial-scm.org/D6694>. We will be needing the nodes which we are removing for later in case of a partial unshelve.
  > The stripbased approach of unshelve failed on the change in D6694 <https://phab.mercurial-scm.org/D6694> on having this change. Should I abandon this?
  
  I think this change is required. A nice way to figure out will be to add the test which you added and not make the change in code and see whether do we remove the extra cset or not.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6686/new/

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

To: navaneeth.suresh, #hg-reviewers
Cc: pulkit, mercurial-devel


More information about the Mercurial-devel mailing list