[darcs-users] [patch452] resolve issue332: ask if test fails

Gabriel Kerneis bugs at darcs.net
Mon Nov 8 16:56:26 UTC 2010


New submission from Gabriel Kerneis <kerneis at pps.jussieu.fr>:

Hi,

this is another attempt to solve issue332, thanks to an idea of Peter
Rockai.

It removes calls to testTentative from finalizeRepositoryChanges and
changes testTentative and testRecorded to return the exit code of the
test.

Each command which accepts the --test (or --no-test) flag is now
responsible for running the test.  When run in interactive mode, if the
test fails, these commands ask the user to confirm the abort.  When run
in non-interactive mode, the old behaviour is kept.

This patch also contains a fix to the darcs testsuite to handle the new
confirmation.

Calls to finalizeRepositoryChanges in Repository.hs (more precisely
those performed by copyPackedRepository and copyNotPackedRepository)
have not been modified because I believe no test is ever run when a full
copy of repository is done.  The same goes for commands like optimize,
etc.

Best,
Gabriel

1 patch for repository http://darcs.net:

Sun Nov  7 20:19:55 CET 2010  Gabriel Kerneis <kerneis at pps.jussieu.fr>
  * resolve issue332: ask if test fails

----------
files: resolve-issue332_-ask-if-test-fails.dpatch, unnamed
messages: 12942
nosy: kerneis
status: needs-screening
title: resolve issue332: ask if test fails

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/patch452>
__________________________________
-------------- next part --------------
A non-text attachment was scrubbed...
Name: resolve-issue332_-ask-if-test-fails.dpatch
Type: text/x-darcs-patch
Size: 74700 bytes
Desc: not available
URL: <http://lists.osuosl.org/pipermail/darcs-users/attachments/20101108/85fad43e/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: unnamed
Type: application/octet-stream
Size: 5 bytes
Desc: not available
URL: <http://lists.osuosl.org/pipermail/darcs-users/attachments/20101108/85fad43e/attachment-0001.obj>


More information about the darcs-users mailing list