[PATCH stable] rebase: derive node from target rev (issue3802)
Pierre-Yves David
pierre-yves.david at ens-lyon.org
Mon Feb 4 14:15:19 CST 2013
On Sun, Feb 03, 2013 at 03:47:29PM -0800, Siddharth Agarwal wrote:
> On 02/03/2013 03:40 PM, Pierre-Yves David wrote:
> >Can't that be just `dest = repo[target]` that would remove needs to update
> >other code part. making the change smaller and safer.
>
> That was the other way to fix this I had in mind. I mildly prefer
> this because that would have one path derive target from dest and
> one derive dest from target, and I thought that was even more
> confusing than this.
meh, given the current state of the rebase code I would have prefered a patch
that change as few as possible of existing code.
The regression here is serious enough.
Kevin seems to have crewed it as is anyway.
--
Pierre-Yves David
More information about the Mercurial-devel
mailing list