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/prefs does.
>   It is named "disttest" because it is analogous to "distclean" inasmuch 
> as it blows away your object files and changes your ./configure.
>   Also remove an obsolete .PHONY target.

This looks good to me.  Is there any reason we shouldn't

darcs setpref test 'make disttest'?

I guess there's the weirdness that running make disttest without first
running autoconf or configure means that the makefile will have to do
that for you prior to running makedisttest.  Also, I wonder if we
shouldn't have

disttest: distclean

just to keep things a bit more reproducible?

David

> hunk ./GNUmakefile 161
>  .PHONY:              all clean distclean maintainer-clean dist deb test 
> check  
> install \
>               test_shell test_shell_hashed test_shell_format2 \
>               test_perl test_perl_hashed test_perl_format2 \
> +             disttest \
>               bugs bugs_shell bugs_shell_hashed bugs_shell_format2 \
>               bugs_perl bugs_perl_hashed bugs_perl_format2 \
>               installserver installdocs installbin windowsinstaller \
> hunk ./GNUmakefile 165
> -             website slowtest witnesses\
> +             website witnesses\
>               api-doc-dir
>
>  config: configure autoconf.mk
> hunk ./GNUmakefile 345
>
>  check test: tests
>
> +disttest:
> +     autoconf && \
> +     ./configure --with-type-witnesses && \
> +     make -j4 witnesses && \
> +     make clean && \
> +     ./configure && \
> +     make -j 4 all unit && \
> +     make -j2 test_unit check test_network
> +
>  tests: \
>       test_shell test_shell_hashed test_shell_format2 \
>       test_perl test_perl_hashed test_perl_format2
_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to