[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