[alsa-devel] [PATCH] Allow opt-out from speex dependency

[Takashi Iwai]

At Thu,  2 Jul 2009 13:42:05 +0200,
Ingmar Vanhassel wrote:
> 
> ---
>  configure.in |    8 +++++---
>  1 files changed, 5 insertions(+), 3 deletions(-)
> 
> This patch makes the --with-speex configure option work as intended, allowing
> to explicitly disable speex linking.

$HAVE_SPEEX should be initialized explicitly before if check.
Otherwise the value is undefined.

Could you fix and repost?


thanks,

Takashi


> 
> diff --git a/configure.in b/configure.in
> index 36740e9..36d0d2a 100644
> --- a/configure.in
> +++ b/configure.in
> @@ -106,14 +106,16 @@ AC_SUBST(AVCODEC_CFLAGS)
>  AC_SUBST(AVCODEC_LIBS)
>  AC_SUBST(AVCODEC_HEADER)
>  
> -PKG_CHECK_MODULES(speexdsp, [speexdsp >= 1.2], [HAVE_SPEEXDSP="yes"], [HAVE_SPEEXDSP=""])
> -AM_CONDITIONAL(HAVE_SPEEXDSP, test "$HAVE_SPEEXDSP" = "yes")
> -
>  AC_ARG_WITH([speex],
>  	AS_HELP_STRING([--with-speex={builtin|lib|no}],
>  		[build speex resampler (built-in code, link with external lib, or no build)]),
>  	[PPH=$withval], [PPH="lib"])
>  
> +if test "x$with_speex" != "xno"; then
> +	PKG_CHECK_MODULES(speexdsp, [speexdsp >= 1.2], [HAVE_SPEEXDSP="yes"], [HAVE_SPEEXDSP=""])
> +fi
> +AM_CONDITIONAL(HAVE_SPEEXDSP, test "$HAVE_SPEEXDSP" = "yes")
> +
>  USE_LIBSPEEX=""
>  if test "$PPH" = "lib"; then
>  	if test "$HAVE_SPEEXDSP" = "yes"; then
> -- 
> 1.6.3.3
> 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>