Re: 23-less-ac-subst.patch

[Pavel Roskin]

On Tue, 23 Jan 2001, Akim Demaille wrote:

> Index: ChangeLog
> from  Akim Demaille  <address@hidden>
>       Don't AC_SUBST too much, as it makes Automake include those
>       variables in Makefiles, and drives autoscan to require unneeded
>       programs.
>
>       * m4sugar.m4 (m4_expand_once): Let the user specify the witness.

Witness? I don't understand why you use this word and how it's supposed to
work.

>       * acgeneral.m4 (_AC_INIT_PREPARE): Don't AC_SUBST CFLAGS,
>       CPPFLAGS, CXXFLAGS and LDFLAGS.
>       (AC_ARG_VAR): AC_SUBST the var.
>       Document it only once, even if there are several different docs.
>       * aclang.m4 (AC_PROG_CPP): Declare CPP and CPPFLAGS to AC_ARG_VAR.
>       (AC_PROG_CXXCPP): Declare CXXCPP and CPPFLAGS.

Ok.

> -# m4_expand_once(TEXT)
> -# --------------------
> -# If TEXT has never been expanded, expand it *here*.
> +# m4_expand_once(TEXT, [WITNESS = TEXT])
> +# --------------------------------------
> +# If TEXT has never been expanded, expand it *here*.  Use WITNESS as
> +# as a memory that TEXT has already been expanded.
>  m4_define([m4_expand_once],
>  [m4_provide_ifelse([$1],
>                     [],

m4_expand_once accepts one more argument but its definition hasn't
changed!

Ok to apply if it's done in a separate patch.

Regards,
Pavel Roskin