I was going to send one bundle to fix all of 
  http://wiki.darcs.net/DefaultSwitches
in time for Darcs 2.5 alpha 1, but maybe I should check first if the general
approach is sane

2 patches for repository http://darcs.net:

Wed Jun  2 18:09:12 BST 2010  Eric Kow <kowey@darcs.net>
  * Make test default more explicit.
  In the context of this patch, the following defaults apply
  
   record : test
   amend-record : test
   check : test
   otherwise : no-test
  
  The aim of this patch is to make the last default (otherwise : no-test)
  more explicit, by making it a forced choice from the Darcs.Repository level
  down by making it a forced choice.  The bigger picture is to move away from
  testing opts membership, which seems fragile.

Wed Jun  2 18:11:00 BST 2010  Eric Kow <kowey@darcs.net>
  * Make --no-test default for record, amend and check.

Attachments

• make-test-default-more-explicit_.dpatch
• unnamed

Make test default more explicit.
--------------------------------
> +data WantTest = WantTest | WantNoTest

It occurred to me on the walk home that here actually, it probably
doesn't make any sense to re-invent Bool.  It was a good idea in lots of
other cases because it avoided passing around a lot of meaningless
looking identical Bools, but since there's little risk of this
happening, it seems like the wantTest function should just return Bool.

> + case wantTest opts of
> +   WantNoTest -> return ()
> +   WantTest   -> withCurrentDirectory dir $

In any case, my more general point was that the default values of flags
should not be buried deep down in the Darcs.Repository level, but should
be closer to the surface so that (A) we can just flip a default without
worrying about breaking some obscure part of the code and (B) we always
know what the defaults are without having to dig through the code.

-- 
Eric Kow <http://www.nltg.brighton.ac.uk/home/Eric.Kow>
PGP Key ID: 08AC04F9

Reviewer: please cherry pick "make --no-test default for record, amend 
and check." and ignore the other patch in this bundle

Hi,

Eric Kow <bugs@darcs.net> writes:
> Reviewer: please cherry pick "make --no-test default for record, amend 
> and check." and ignore the other patch in this bundle
OK, applied, thanks. (Just the latter patch. I guess accepted is
reasonable in this case.)

Yours,
   Petr.