From octave-maintainers-request at bevo dot che dot wisc dot edu  Sun May  6 14:28:40 2001
Subject: Re: [patch] Use fftw if available
From: "Steven G. Johnson" <stevenj at superjdj dot mit dot edu>
To: octave-maintainers at bevo dot che dot wisc dot edu
cc: Mumit Khan <khan at NanoTech dot Wisc dot EDU>
Date: Sun, 6 May 2001 15:28:10 -0400 (EDT)

JWE wrote:
> I applied your patches and made the default configure action to check
> for and use the fftw library even if --with-fftw is not specified.

Hi, note that AC_ARG_WITH is being used incorrectly in configure.in. Since
the default is now --with-fftw, then the help message should say
--without-fftw, which is the command to turn it off.  Also, you need to
use $withval so that the user can turn it off or on.  The patch below
fixes this.

Steven

Index: configure.in
===================================================================
RCS file: /cvs/octave/configure.in,v
retrieving revision 1.347
diff -u -r1.347 configure.in
--- configure.in	2001/05/02 06:15:07	1.347
+++ configure.in	2001/05/06 19:24:05
 at @ -362,8 +362,8 @@
 AC_SUBST(FFTW_LIBS)
 
 AC_ARG_WITH(fftw, 
-  [  --with-fftw             use installed fftw instead of included fftpack],
-  with_fftw=yes, with_fftw=yes)
+  [  --without-fftw          use included fftpack instead of installed fftw],
+  with_fftw=$withval, with_fftw=yes)
 
 if test "$with_fftw" = "yes"; then
   AC_CHECK_HEADERS(dfftw.h fftw.h)