D6686: unshelve: handle stripping changesets on interactive mode
navaneeth.suresh (Navaneeth Suresh)
phabricator at mercurial-scm.org
Fri Jul 26 11:22:40 EDT 2019
navaneeth.suresh added a comment.
In D6686#98006 <https://phab.mercurial-scm.org/D6686#98006>, @pulkit wrote:
> 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.
Yes, this change is required. D6694 <https://phab.mercurial-scm.org/D6694> is modified and rebased on top of this change.
CHANGES SINCE LAST ACTION
To: navaneeth.suresh, #hg-reviewers
Cc: pulkit, mercurial-devel
More information about the Mercurial-devel