Re: [bug-gawk] build error with './configure --disable-extensions'

[arnold]

Hi.

Thanks for the report. That looks like the right fix. I will test it and
most likely apply it.

Thanks!

Arnold


Assaf Gordon <address@hidden> wrote:

> Hello,
>
> I think there might be a problem in gawk's build when using 
> --disable-extensions.
>
> With the latest release:
>
>   $ wget http://ftp.gnu.org/gnu/gawk/gawk-4.1.3.tar.xz
>   $ tar -xf gawk-4.1.3.tar.xz 
>   $ cd gawk-4.1.3/
>   $ ./configure --disable-extensions
>   $ make
>   [...]
>   Making all in po
>   make[2]: Nothing to be done for `all'.
>   Making all in extension
>   make[2]: *** No rule to make target `all'.  Stop.
>   make[1]: *** [all-recursive] Error 1
>   make: *** [all] Error 2
>
> And with the latest git repository:
>
>   $ git describe
>   gawk-4.1.0-1818-g4f75877
>
>   $ ./configure --disable-extensions 
>   [...]
>   configure: error: conditional "ENABLE_EXTENSIONS" was never defined.
>   Usually this means the macro was only invoked conditionally.
>
> This might be because "AM_CONDITIONAL([ENABLE_EXTENSIONS]" is defined inside 
> an 'if'.
>
> The following might resolve it, but I have not tested it thoroughly.
>
> regards,
>  -assaf
>
> ====
>
> diff --git a/configure.ac b/configure.ac
> index a3e41e5..bfe785d 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -315,8 +315,8 @@ if test "x$enable_extensions" != "xno"; then
>                 AC_MSG_ERROR([extension support requested, but unavailable])
>         fi
>         enable_extensions=$extensions_supported
> -       AM_CONDITIONAL([ENABLE_EXTENSIONS], [test "x$enable_extensions" = 
> "xyes"])
>  fi
> +AM_CONDITIONAL([ENABLE_EXTENSIONS], [test "x$enable_extensions" = "xyes"])
>  
>  dnl check for how to use getpgrp
>  dnl have to hardwire it for VMS POSIX. Sigh.
>
>
>