[PATCH 5 of 5] histedit: do not stay on a cleaned nodes on abort

Martin von Zweigbergk martinvonz at google.com
Wed Aug 5 12:03:14 CDT 2015


This looks good and simple enough even for someone not familiar with
histedit internals that I dare push it to the clowncopter (done). Thanks!

On Mon, Aug 3, 2015 at 5:29 PM Laurent Charignon <lcharignon at fb.com> wrote:

> Oh yeah, sorry I forgot that we would need that extension to add the test.
> You can proceed then.
>
> Thanks,
>
> Laurent
>
> > On Aug 3, 2015, at 4:35 PM, Pierre-Yves David <
> pierre-yves.david at ens-lyon.org> wrote:
> >
> >
> >
> > On 08/03/2015 04:29 PM, Laurent Charignon wrote:
> >> This series looks good to me and seems to fix the issues that we have
> been seeing using histedit with evolve.
> >> I talked to you offline about a test case that used to not work and
> that this series is fixing, let's add it to make sure that we don't regress.
> >
> > The test case required the histedit extensions use at Facebook right?
> and so the test case should live there. Or are we talking about something
> else?
> >
> > --
> > Pierre-Yves David
>
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> https://selenic.com/mailman/listinfo/mercurial-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20150805/ca9e26a2/attachment.html>


More information about the Mercurial-devel mailing list