ACLOCAL_AMFLAGS: fully deprecate, with warnings in the 'obsolete' category

Users are expected to use the AC_CONFIG_MACRO_DIR{,S} autoconf macros
instead.  And the use of the '--install' aclocal option in ACLOCAL_AMFLAGS
has proved to be a bad idea anyway -- see automake bug#9037.

* NEWS: Update.
* t/aclocal-amflags.sh: Enhance and adjust.
* t/acloca14b.sh: Adjust.
* t/acloca22b.sh: Likewise.
* t/ax/distcheck-hook-m4.am: Likewise.
* t/dist-missing-m4.sh: Likewise.
* t/remake-renamed-m4-macro.sh: Likewise.
* t/remake-renamed-m4-macro-and-file.sh: Likewise.
* t/remake-renamed-m4-file.sh: Likewise.
* t/remake-moved-m4-file.sh: Likewise.
* t/remake-gnulib-remove-header.sh: Likewise.
* t/remake-gnulib-add-acsubst.sh: Likewise.
* t/remake-deleted-m4-file.sh: Likewise.
* t/remake-deleted-m4-file.sh: Likewise.
* t/libtool-macros.sh: Likewise.
* t/gettext-macros.sh: Likewise.

Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>

diff --git a/NEWS b/NEWS
index a8b2cdd7fdeb691a5880de317d363172044dad35..2d63043052c93fcfe20a69b87214c62e244b9430 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -11,11 +11,18 @@ New in 1.14:
     over the same-named automake-provided macro, as defined in file
     '/usr/local/share/aclocal-1.14/vala.m4'.
 
-* Obsolete features:
+* Obsolete and obsolescent features:
 
   - Support for the long-obsolete $(INCLUDES) variable has been finally
     removed, in favour of the modern equivalent $(AM_CPPFLAGS).
 
+  - 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.  And use of the '--install' aclocal option in
+    'ACLOCAL_AMFLAGS' has proved to be a bad idea anyway -- see automake
+    bug#9037.
+
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 New in 1.13:

diff --git a/automake.in b/automake.in
index 96ff685656987096e8a55c650907ce8a7637b6ea..ffe16e22c1211484e23876dbf1ae2ad45972d23c 100644 (file)
--- a/automake.in
+++ b/automake.in
@@ -7832,6 +7832,12 @@ sub generate_makefile ($$)
   # Catch some obsolete variables.
   reject_var 'INCLUDES', "'INCLUDES' is the obsolete name for " .
                          "'AM_CPPFLAGS' (or '*_CPPFLAGS')";
+  if (my $amflags = var ('ACLOCAL_AMFLAGS'))
+    {
+      msg_var 'obsolete', $amflags,
+              "'ACLOCAL_AMFLAGS' is deprecated; use 'AC_CONFIG_MACRO_DIRS'"
+              . " in configure.ac instead";
+    }
 
   # Must do this after reading .am file.
   define_variable ('subdir', $relative_dir, INTERNAL);

diff --git a/t/aclocal-amflags.sh b/t/aclocal-amflags.sh
index 49bcf5f373ac4bccf5e1f880e94902aebbefefcb..69bce3201bde11df05125f8cfd63b1afba2a9277 100755 (executable)
--- a/t/aclocal-amflags.sh
+++ b/t/aclocal-amflags.sh
@@ -15,8 +15,8 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 # Check that the obsolescent idiom of setting $(ACLOCAL_AMFLAGS) in
-# Makefile.am still works.  Remove this test once support for this
-# obsolescent idiom is removed.
+# Makefile.am is warned against, but still works.  Remove this test
+# once support for this obsolescent idiom is removed.
 
 . test-init.sh
 
@@ -45,7 +45,16 @@ $ACLOCAL -I m4_1 >output 2>&1 || { cat output; exit 1; }
 cat output
 grep 'found macro' output && exit 1 # Sanity check.
 $AUTOCONF
-$AUTOMAKE
+
+for opts in '' '-Wnone -Wobsolete'; do
+  AUTOMAKE_fails $opts
+  grep "Makefile\.am:.*'ACLOCAL_AMFLAGS' is deprecated" stderr
+  grep "Makefile\.am:.*'AC_CONFIG_MACRO_DIRS'.*configure\.ac.*instead" stderr
+done
+
+echo AUTOMAKE_OPTIONS = -Wall -Wno-obsolete >> Makefile.am
+cat Makefile.am # For debugging.
+$AUTOMAKE -Werror
 
 ./configure
 test -f foo

diff --git a/t/aclocal-deleted-header-aclocal-amflags.sh b/t/aclocal-deleted-header-aclocal-amflags.sh
index ea47635c0b7251aac0f90f546fa178f3d7618082..8f11ebdd5cff0d8ccbe626731729f92f99f3212b 100755 (executable)
--- a/t/aclocal-deleted-header-aclocal-amflags.sh
+++ b/t/aclocal-deleted-header-aclocal-amflags.sh
@@ -22,7 +22,10 @@
 
 . test-init.sh
 
-cat >>configure.ac <<EOF
+cat > configure.ac <<EOF
+AC_INIT([$me], [1.3])
+AM_INIT_AUTOMAKE([-Wno-obsolete])
+AC_CONFIG_FILES([Makefile])
 FOO
 AC_OUTPUT
 EOF

diff --git a/t/aclocal-m4-include-are-scanned-aclocal-amflags.sh b/t/aclocal-m4-include-are-scanned-aclocal-amflags.sh
index 31a05409b1f975722f608a32cdeda8aa340dbc1f..c7b23924ddbcbc55be16cc8fe19071aa132e4a4a 100755 (executable)
--- a/t/aclocal-m4-include-are-scanned-aclocal-amflags.sh
+++ b/t/aclocal-m4-include-are-scanned-aclocal-amflags.sh
@@ -46,6 +46,7 @@ echo 'AC_DEFUN([ANOTHER_MACRO])' >defs/f.m4
 
 cat >>Makefile.am<<\EOF
 ACLOCAL_AMFLAGS = -I defs
+AUTOMAKE_OPTIONS = -Wno-obsolete
 testdist1: distdir
        test -f $(distdir)/acinclude.m4
        test -f $(distdir)/a.m4

diff --git a/t/ax/distcheck-hook-m4.am b/t/ax/distcheck-hook-m4.am
index 5d36102c6cffa90618ef02ba1a0b74c45eb1229d..262e370c95726ee1cf4fcab3bb3c1e4a0d4543c0 100644 (file)
--- a/t/ax/distcheck-hook-m4.am
+++ b/t/ax/distcheck-hook-m4.am
@@ -15,6 +15,8 @@
 ## You should have received a copy of the GNU General Public License
 ## along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
+AUTOMAKE_OPTIONS = -Wno-obsolete
+
 distcheck-hook:
        @fatal () { echo "$@: $$*" >&2; exit 1; }; \
        $(am__cd) $(distdir) && chmod u+w . && mkdir _m4 \

diff --git a/t/dist-missing-m4.sh b/t/dist-missing-m4.sh
index e0ed88614ee7b7b30732d941848773a17ba22763..c5eed848b31eee28612d875713cd7c70e3da991f 100755 (executable)
--- a/t/dist-missing-m4.sh
+++ b/t/dist-missing-m4.sh
@@ -23,6 +23,7 @@
 . test-init.sh
 
 cat >> configure.ac <<'END'
+AC_CONFIG_MACRO_DIR([m4])
 m4_pattern_forbid([^MY_])
 MY_FOOBAR || exit 1
 MY_ZARDOZ || exit 1
@@ -33,9 +34,9 @@ mkdir m4
 echo 'AC_DEFUN([MY_FOOBAR], [:])' > m4/foobar.m4
 echo 'AC_DEFUN([MY_ZARDOZ], [:])' > m4/zardoz.m4
 
-echo 'ACLOCAL_AMFLAGS = -I m4' > Makefile.am
+: > Makefile.am
 
-$ACLOCAL -I m4
+$ACLOCAL
 $AUTOCONF
 $AUTOMAKE
 

diff --git a/t/gettext-macros.sh b/t/gettext-macros.sh
index 6d58c9c37f4e33218a490d19ebfa2281eaa83435..17045c6112d5b18a14a3f211774e135889e2c7e6 100755 (executable)
--- a/t/gettext-macros.sh
+++ b/t/gettext-macros.sh
@@ -29,7 +29,8 @@ echo "# Automatically generated by $me." > get.sh
 echo : >> get.sh
 
 # The 'gettextize' and 'autopoint' scripts will look into Makefile.am.
-echo ACLOCAL_AMFLAGS = -I m4 > Makefile.am
+echo AUTOMAKE_OPTIONS = -Wno-obsolete > Makefile.am
+echo ACLOCAL_AMFLAGS = -I m4 >> Makefile.am
 
 # Prefer autopoint to gettextize, since the more modern versions of the
 # latter might unconditionally require user interaction to complete;

diff --git a/t/libtool-macros.sh b/t/libtool-macros.sh
index c50777b7cb4de4095a36b19d0edbf2ed4093ffa0..ec84682a1454b2222206f976dfac5a4e18a866bf 100755 (executable)
--- a/t/libtool-macros.sh
+++ b/t/libtool-macros.sh
@@ -23,8 +23,7 @@
 echo "# Automatically generated by $me." > get.sh
 echo : >> get.sh
 
-# The 'libtoolize' script will look into Makefile.am.
-echo ACLOCAL_AMFLAGS = -I m4 > Makefile.am
+echo 'AC_CONFIG_MACRO_DIR([m4])' >> configure.ac
 
 if libtoolize --copy --install && test -f m4/libtool.m4; then
   echo "ACLOCAL_PATH='$(pwd)/m4':\$ACLOCAL_PATH" >> get.sh
@@ -37,7 +36,7 @@ else
   mkdir m4
   echo AC_PROG_LIBTOOL >> configure.ac
   # See below for an explanation about the use the of '-Wno-syntax'.
-  if $ACLOCAL -Wno-syntax -I m4 --install && test -f m4/libtool.m4; then
+  if $ACLOCAL -Wno-syntax --install && test -f m4/libtool.m4; then
     : # Libtool macros already accessible by default.
   else
     echo "skip_all_ \"couldn't find or get libtool macros\"" >> get.sh

diff --git a/t/remake-deleted-m4-file.sh b/t/remake-deleted-m4-file.sh
index 2499d82cdba14fc8aaab8560c4b34c4a00e5e962..d733a56c5d4c32423005573445be01527166b009 100755 (executable)
--- a/t/remake-deleted-m4-file.sh
+++ b/t/remake-deleted-m4-file.sh
@@ -25,10 +25,11 @@
 cat >> configure.ac <<'END'
 FOO_MACRO
 AC_OUTPUT
+dnl Deliberately put this late, for better coverage.
+AC_CONFIG_MACRO_DIR([m4])
 END
 
 cat > Makefile.am <<'END'
-ACLOCAL_AMFLAGS = -I m4
 .PHONY: test
 test:
        test '$(the_answer)' -eq 42
@@ -46,7 +47,7 @@ cat > m4/bar.m4 <<END
 AC_DEFUN([BAR_MACRO], [$macro_value])
 END
 
-$ACLOCAL -I m4
+$ACLOCAL
 $AUTOCONF
 $AUTOMAKE
 

diff --git a/t/remake-gnulib-add-acsubst.sh b/t/remake-gnulib-add-acsubst.sh
index 3aea5b9985af025960629d41d308cdf6e1c02f50..4cfad964a59371bed259f11125780965277c0d72 100755 (executable)
--- a/t/remake-gnulib-add-acsubst.sh
+++ b/t/remake-gnulib-add-acsubst.sh
@@ -26,12 +26,12 @@ required=cc
 
 cat >> configure.ac <<'END'
 AC_PROG_CC
+AC_CONFIG_MACRO_DIRS([m4])
 MY_MACROS
 AC_OUTPUT
 END
 
 cat > Makefile.am <<'END'
-ACLOCAL_AMFLAGS = -I m4
 noinst_PROGRAMS = foo
 foo_SOURCES = foo.c
 BUILT_SOURCES = foo.h
@@ -70,7 +70,7 @@ int main (void) { return 0; }
 typedef int checkfoo[1 - 2 * (foo != 42)];
 END
 
-$ACLOCAL -I m4
+$ACLOCAL
 $AUTOCONF
 $AUTOMAKE
 

diff --git a/t/remake-gnulib-add-header.sh b/t/remake-gnulib-add-header.sh
index c70b3c672548414906864279034664884a4963f6..db6bea1869181077aea13490ca7a4bc44e33f9f5 100755 (executable)
--- a/t/remake-gnulib-add-header.sh
+++ b/t/remake-gnulib-add-header.sh
@@ -25,6 +25,7 @@ required=cc
 . test-init.sh
 
 cat >> configure.ac <<'END'
+AC_CONFIG_MACRO_DIR([.])
 AC_CONFIG_HEADERS([config.h])
 AC_PROG_CC
 MY_MACROS
@@ -32,7 +33,6 @@ AC_OUTPUT
 END
 
 cat > Makefile.am <<'END'
-ACLOCAL_AMFLAGS = -I .
 noinst_PROGRAMS = foo
 foo_SOURCES = foo.c
 BUILT_SOURCES = $(STDIO_H)
@@ -80,7 +80,7 @@ FILE *f;
 int main () { return 0; }
 END
 
-$ACLOCAL -I .
+$ACLOCAL
 $AUTOHEADER
 $AUTOMAKE
 $AUTOCONF

diff --git a/t/remake-gnulib-remove-header.sh b/t/remake-gnulib-remove-header.sh
index 23fbfc81bfe4d91642eeb2c5459ccdbcab2d4e37..189bbe5d31a245cad8ff603cc3212cb2a31c836d 100755 (executable)
--- a/t/remake-gnulib-remove-header.sh
+++ b/t/remake-gnulib-remove-header.sh
@@ -26,6 +26,7 @@ required=cc
 . test-init.sh
 
 cat >> configure.ac <<'END'
+AC_CONFIG_MACRO_DIR([.])
 AC_CONFIG_HEADERS([config.h])
 AC_PROG_CC
 MY_MACROS
@@ -33,7 +34,6 @@ AC_OUTPUT
 END
 
 cat > Makefile.am <<'END'
-ACLOCAL_AMFLAGS = -I .
 noinst_PROGRAMS = foo
 foo_SOURCES = foo.c
 BUILT_SOURCES = $(STDIO_H)
@@ -79,7 +79,7 @@ FILE *f;
 int main () { return 0; }
 END
 
-$ACLOCAL -I .
+$ACLOCAL
 $AUTOHEADER
 $AUTOMAKE
 $AUTOCONF

diff --git a/t/remake-moved-m4-file.sh b/t/remake-moved-m4-file.sh
index 3ca16801fe9968c0ff83bb3f4ddcd86f97e3a806..faed95d8b956cbecc97e21d8f7e985d5b3428aa3 100755 (executable)
--- a/t/remake-moved-m4-file.sh
+++ b/t/remake-moved-m4-file.sh
@@ -21,11 +21,12 @@
 
 cat >> configure.ac <<'END'
 MY_MACRO
+AC_CONFIG_MACRO_DIRS([d1])
+AC_CONFIG_MACRO_DIRS([d2 d3])
 AC_OUTPUT
 END
 
 cat > Makefile.am <<'END'
-ACLOCAL_AMFLAGS = -I d1 -I d2 -I d3
 .PHONY: test
 test:
        test '$(the_answer)' -eq 42
@@ -41,7 +42,7 @@ cat > d1/foo.m4 <<'END'
 AC_DEFUN([FOO], [the_answer=42; AC_SUBST([the_answer])])
 END
 
-$ACLOCAL -I d1
+$ACLOCAL
 $AUTOCONF
 $AUTOMAKE
 

diff --git a/t/remake-renamed-m4-file.sh b/t/remake-renamed-m4-file.sh
index ff99f1d54a4b2ed44ca1d9573db3ee81d1c598e1..462a2ce01ab9e8bd47f55f40471027edc81462e1 100755 (executable)
--- a/t/remake-renamed-m4-file.sh
+++ b/t/remake-renamed-m4-file.sh
@@ -19,12 +19,12 @@
 . test-init.sh
 
 cat >> configure.ac <<'END'
+AC_CONFIG_MACRO_DIR([m4])
 MY_MACRO
 AC_OUTPUT
 END
 
 cat > Makefile.am <<'END'
-ACLOCAL_AMFLAGS = -I m4
 .PHONY: test
 test:
        test '$(the_answer)' -eq 42
@@ -40,7 +40,7 @@ cat > m4/foo.m4 <<'END'
 AC_DEFUN([FOO], [the_answer=42; AC_SUBST([the_answer])])
 END
 
-$ACLOCAL -I m4
+$ACLOCAL
 $AUTOCONF
 $AUTOMAKE
 

diff --git a/t/remake-renamed-m4-macro-and-file.sh b/t/remake-renamed-m4-macro-and-file.sh
index 9658737096ab08bd88e11f387dfe9853ebf772ef..985f5c67acab1f31affadaf928502259ebd7cd46 100755 (executable)
--- a/t/remake-renamed-m4-macro-and-file.sh
+++ b/t/remake-renamed-m4-macro-and-file.sh
@@ -21,12 +21,12 @@
 . test-init.sh
 
 cat >> configure.ac <<'END'
+AC_CONFIG_MACRO_DIR([m4])
 MY_MACRO
 AC_OUTPUT
 END
 
 cat > Makefile.am <<'END'
-ACLOCAL_AMFLAGS = -I m4
 .PHONY: test
 test:
        test '$(the_answer)' -eq 42
@@ -42,7 +42,7 @@ cat > m4/foo.m4 <<'END'
 AC_DEFUN([FOO_MACRO], [the_answer=42; AC_SUBST([the_answer])])
 END
 
-$ACLOCAL -I m4
+$ACLOCAL
 $AUTOCONF
 $AUTOMAKE
 

diff --git a/t/remake-renamed-m4-macro.sh b/t/remake-renamed-m4-macro.sh
index 99ab1db90c6fbce9e2b6d798772349e3ad60d9dc..0d97a221e80648b1509cbc8d923ec8a4935ba1aa 100755 (executable)
--- a/t/remake-renamed-m4-macro.sh
+++ b/t/remake-renamed-m4-macro.sh
@@ -20,12 +20,12 @@
 . test-init.sh
 
 cat >> configure.ac <<'END'
+AC_CONFIG_MACRO_DIR([m4])
 MY_MACRO
 AC_OUTPUT
 END
 
 cat > Makefile.am <<'END'
-ACLOCAL_AMFLAGS = -I m4
 .PHONY: test
 test:
        test '$(the_answer)' -eq 42
@@ -42,7 +42,7 @@ AC_DEFUN([FOO_1], [the_answer=42
                    AC_SUBST([the_answer])])
 END
 
-$ACLOCAL -I m4
+$ACLOCAL
 $AUTOCONF
 $AUTOMAKE