bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: gnulib-tool uses deprecated ACLOCAL_AMFLAGS


From: Bruno Haible
Subject: Re: gnulib-tool uses deprecated ACLOCAL_AMFLAGS
Date: Mon, 07 Nov 2016 17:40:06 +0100
User-agent: KMail/4.8.5 (Linux/3.8.0-44-generic; KDE/4.8.5; x86_64; ; )

Hi,

Michael Stapelberg wrote:
> ACLOCAL_AMFLAGS was deprecated in 2012 according to
> https://lists.gnu.org/archive/html/automake-patches/2012-12/msg00079.html

What should the replacement be? The Automake NEWS from git (unreleased) says:

  - Use of the special makefile variable 'ACLOCAL_AMFLAGS' is deprecated.
    To specify locations of extra m4 files, the 'AC_CONFIG_MACRO_DIR' or
    'AC_CONFIG_MACRO_DIRS' (the latter introduced with autoconf 2.70)
    should be used instead.

The Autoconf NEWS from git (unreleased) goes into more details:

  - AC_CONFIG_MACRO_DIRS
    New macro, used to declare multiple directories when looking for
    local M4 macros.  This macro overcomes some of the shortfalls in the
    older AC_CONFIG_MACRO_DIR, which could only be used once; ...

The result of gnulib-tool most often contains a directory m4 files which
is not the only one for a given package. Therefore the recommended replacement
should be to use AC_CONFIG_MACRO_DIRS.

But AC_CONFIG_MACRO_DIRS is not yet released.

> Yet, the gnulib git repository contains a couple of references to it (as of
> commit 2f8b355aeacb448d13bab9cb3cdaaeea649a4664):
> 
> $ grep -r ACLOCAL_AMFLAGS
> gnulib-tool:    echo "ACLOCAL_AMFLAGS = -I ${testsbase_inverse}/${m4base}"
> gnulib-tool:  func_note_Makefile_am_edit "" ACLOCAL_AMFLAGS "-I ${m4base}"
> gnulib-tool:   echo "ACLOCAL_AMFLAGS = -I $m4base"
> gnulib-tool:          aclocal_amflags=`sed -n -e 's/^ACLOCAL_AMFLAGS[
> ]*=\(.*\)$/\1/p'
> "$destdir"/Makefile.am`
> doc/gnulib-tool.texi:  - mention "-I m4" in ACLOCAL_AMFLAGS in Makefile.am,
> doc/gnulib-tool.texi:@file{gnulib-comp.m4}.  Use the @code{ACLOCAL_AMFLAGS}
> specifier in
> doc/gnulib-tool.texi:ACLOCAL_AMFLAGS = -I m4
> ChangeLog: ACLOCAL_AMFLAGS parsing state machine.
> 
> I originally intended to just send a patch to update the documentation, but
> now I’m not certain if that in itself is a good idea without also updating
> the code, and I don’t have the time to wrestle with large shell scripts
> currently :).

This is all welcome, but only after Autoconf 2.70 is released.

Bruno




reply via email to

[Prev in Thread] Current Thread [Next in Thread]