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.
CHANGES SINCE LAST ACTION
To: navaneeth.suresh, #hg-reviewers
Cc: pulkit, mercurial-devel
More information about the Mercurial-devel