[PATCH 1/1] added configuration option for building tests

[Trevor Saunders]

Hi,

The code looks fine, but I have two questions.  where is the commit
message? :-)  and why do we install these in the first place?  shouldn't
they only be used in make check?

Trev

On Wed, Sep 08, 2010 at 04:44:36PM +0200, Andrei Kholodnyi wrote:
> ---
>  configure.in    |    8 ++++++++
>  src/Makefile.am |    6 +++++-
>  2 files changed, 13 insertions(+), 1 deletions(-)
> 
> diff --git a/configure.in b/configure.in
> index d0e31b8..cfb65a1 100644
> --- a/configure.in
> +++ b/configure.in
> @@ -18,6 +18,11 @@ AC_ARG_ENABLE([python],
>       [],
>       [enable_python=check])
>  
> +AC_ARG_ENABLE([tests],
> +     [AS_HELP_STRING([--disable-tests], [do not install tests])],
> +     [],
> +     [enable_tests=yes])
> +
>  dnl Checks for header files.
>  AC_HEADER_STDC
>  AC_CHECK_HEADERS(limits.h sys/ioctl.h sys/time.h unistd.h)
> @@ -35,6 +40,9 @@ AS_IF([test $enable_python != "no"],
>               [AC_MSG_FAILURE([python is not available])])])])
>  AM_CONDITIONAL([HAVE_PYTHON], [test "$enable_python" = 'yes'])
>  
> +# check for tests support
> +AM_CONDITIONAL(tests_support, test $enable_tests = "yes")
> +
>  dnl Checks for library functions.
>  AC_PROG_GCC_TRADITIONAL
>  AC_CHECK_FUNCS(select socket)
> diff --git a/src/Makefile.am b/src/Makefile.am
> index fa6a695..9f34ce1 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -10,12 +10,16 @@ EXTRA_DIST = cl/ChangeLog cl/README cl/configuration.lisp 
> cl/elisp.lisp \
>               guile/ChangeLog guile/Makefile guile/README guile/gssip.scm.in \
>               guile/gssip.c guile/gssip.h
>  
> -SUBDIRS=server audio c modules tests
> +SUBDIRS=server audio c modules
>  
>  if HAVE_PYTHON
>  SUBDIRS += python
>  endif
>  
> +if tests_support
> +SUBDIRS += tests
> +endif
> +
>  DIST_SUBDIRS=server audio c modules tests python
>  
>  
> -- 
> 1.7.0.4
> 
> 
> _______________________________________________
> Speechd mailing list
> Speechd at lists.freebsoft.org
> http://lists.freebsoft.org/mailman/listinfo/speechd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: 
<http://lists.freebsoft.org/pipermail/speechd/attachments/20100908/8226904f/attachment.pgp>