[darcs-devel] darcs patch: add test for replace that messes with unrecorded hunks

David Roundy droundy at darcs.net
Sat Jan 27 15:36:45 PST 2007


On Thu, Jan 25, 2007 at 06:25:05PM +0100, ptp at lysator.liu.se wrote:
> This test makes current unstable fail with invalid pending bug.
> Trackdown finds:
> 
> |  Sat Dec  9 23:39:16 CET 2006  David Roundy <droundy at darcs.net>
> |    * fix pending bug that broke several_commands.sh.
> 
> Removing it and its dependencies makes the test pass.

Thanks for the test! I'm looking into this right now, and it turns out to
be a little bit tricky.  (Which is why I'm sending this note before a
patch.)  I've got a fix, but decided to do a refactor first.

> I'm not sure the test _should_ pass without the --force flag,
> but in that case 1.0.8 is broken as well.

Yeah, it should pass, because we only should need --force if the changes
we want to deal with are already recorded.
-- 
David Roundy
http://www.darcs.net


More information about the darcs-devel mailing list