[darcs-devel] darcs patch: test: Exibit a falling test about rollback.

Nicolas Pouillard nicolas.pouillard at gmail.com
Thu Jan 10 17:23:56 UTC 2008


Excerpts from David Roundy's message of Thu Jan 10 18:08:59 +0100 2008:
> On Mon, Jan 07, 2008 at 03:43:40PM +0000, Nicolas Pouillard wrote:
> > Mon Jan  7 16:02:24 CET 2008  nicolas.pouillard at gmail.com
> >   * test: Exibit a falling test about rollback.
> >   Indeed  the only test about rollback was br0ken by a prior test that creates a
> >   directory  and  remove  read  permissions  to  it.  The  rollback test do some
> >   records  that  silently  fail  by lack of permissions, finally the rollback is
> >   cancelled since the named patch doesn't exist.
> >   This shows that rollback need some care.
> 
> Thanks for the patch!
> 
> I've actually been debating the idea of removing the rollback command.
> It's poorly implemented, and has been a source of confusion and problems.
> What do you think?

The source of problems was about hidden conflicts, right?
It's no longer a problem in darcs2, right?

It's mainly a common use case when can no longer use amend-record.

I  think that's also a great tool to temporarily revert a patch without having
two repositories.

Moreover  this  kind of operation is waited when one know that patches must be
invertible.

-- 
Nicolas Pouillard aka Ertai


More information about the darcs-devel mailing list