Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-05 Thread Eric Kow
On Fri, Sep 05, 2008 at 18:56:00 +1000, Trent W. Buck wrote: > Perhaps a better solution would be to have the test rule start by > printing > > To record without testing, add --no-test. This sounds like it might work. I have sent some patches to implement this. Thanks! -- Eric Kow

Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-05 Thread Trent W. Buck
"Eric Y. Kow" <[EMAIL PROTECTED]> writes: > On Mon, Sep 01, 2008 at 17:55:03 -0400, David Roundy wrote: >> The maintainers can certainly use different settings for the test, but >> I don't think there's a test that's much faster that will really help >> significantly--except that we could skip the

Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-04 Thread David Roundy
On Tue, Sep 2, 2008 at 5:16 AM, Eric Y. Kow <[EMAIL PROTECTED]> wrote: > On Mon, Sep 01, 2008 at 17:55:03 -0400, David Roundy wrote: >> The maintainers can certainly use different settings for the test, but >> I don't think there's a test that's much faster that will really help >> significantly--e

Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-03 Thread Eric Y. Kow
On Mon, Sep 01, 2008 at 17:55:03 -0400, David Roundy wrote: > The maintainers can certainly use different settings for the test, but > I don't think there's a test that's much faster that will really help > significantly--except that we could skip the unit tests, which involve > code that is almost

Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-01 Thread David Roundy
On Mon, Sep 01, 2008 at 10:45:24PM +0100, Eric Y. Kow wrote: > On Mon, Sep 01, 2008 at 15:40:31 -0600, zooko wrote: > > On Sep 1, 2008, at 12:22 PM, David Roundy wrote: > > > > > This looks good to me. Is there any reason we shouldn't > > > > > > darcs setpref test 'make disttest'? > > I'll have

Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-01 Thread Eric Y. Kow
On Mon, Sep 01, 2008 at 15:40:31 -0600, zooko wrote: > On Sep 1, 2008, at 12:22 PM, David Roundy wrote: > > > This looks good to me. Is there any reason we shouldn't > > > > darcs setpref test 'make disttest'? I'll have to confess that I tend to record --no-test. It makes me feel guilty, but I

Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-01 Thread David Roundy
On Mon, Sep 01, 2008 at 03:40:31PM -0600, zooko wrote: > On Sep 1, 2008, at 12:22 PM, David Roundy wrote: > >> This looks good to me. Is there any reason we shouldn't >> >> darcs setpref test 'make disttest'? > > Not that I can think of. > >> disttest: distclean >> >> just to keep things a bit mor

Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-01 Thread zooko
On Sep 1, 2008, at 12:22 PM, David Roundy wrote: > This looks good to me. Is there any reason we shouldn't > > darcs setpref test 'make disttest'? Not that I can think of. > disttest: distclean > > just to keep things a bit more reproducible? That sounds like a good idea, too. Regards, Zooko

Re: [darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-01 Thread David Roundy
On Mon, Sep 01, 2008 at 10:44:19AM -0600, zooko wrote: > Mon Sep 1 10:43:42 MDT 2008 [EMAIL PROTECTED] > * add makefile target "disttest" which does the script currently > stored in _darcs/prefs/prefs > This way other things such as buildbot can easily run the same tests as > _darcs/prefs/

[darcs-users] patch: add makefile target named "disttest" which does the test currently written in _darcs/prefs/prefs

2008-09-01 Thread zooko
Mon Sep 1 10:43:42 MDT 2008 [EMAIL PROTECTED] * add makefile target "disttest" which does the script currently stored in _darcs/prefs/prefs This way other things such as buildbot can easily run the same tests as _darcs/prefs/prefs does. It is named "disttest" because it is analogous to