From: Akim Demaille <akim@epita.fr>
Date: Fri, 3 Nov 2000 16:02:54 +0000 (+0000)
Subject: AC_CONFIG_AUX_DIR_DEFAULTS overrides AC_CONFIG_AUX_DIR.  Ouch.
X-Git-Tag: autoconf-2.50~463
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b4a8fae52bfa9fdc1132ee025ed17988f74169b0;p=thirdparty%2Fautoconf.git

AC_CONFIG_AUX_DIR_DEFAULTS overrides AC_CONFIG_AUX_DIR.  Ouch.
Reported by Paul Martinolich.
* acgeneral.m4 (AC_PROVIDE): Don't forget to `m4_provide'!!!
Tss, novice...
* tests/base.at (AC_REQUIRE & AC_PROVIDE): New test.
---

diff --git a/ChangeLog b/ChangeLog
index 838468274..98c7a226c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2000-11-03  Akim Demaille  <akim@epita.fr>
+
+	AC_CONFIG_AUX_DIR_DEFAULTS overrides AC_CONFIG_AUX_DIR.  Ouch.
+	Reported by Paul Martinolich.
+
+	* acgeneral.m4 (AC_PROVIDE): Don't forget to `m4_provide'!!!
+	Tss, novice...
+	* tests/base.at (AC_REQUIRE & AC_PROVIDE): New test.
+
 2000-11-03  Akim Demaille  <akim@epita.fr>
 
 	* m4sugar.m4 (m4_location): When using its value, don't use
diff --git a/acgeneral.m4 b/acgeneral.m4
index 7ae5b45d6..eadea64b2 100644
--- a/acgeneral.m4
+++ b/acgeneral.m4
@@ -264,10 +264,11 @@ m4_define([AC_EXPAND_ONCE],
 
 # AC_PROVIDE(MACRO-NAME)
 # ----------------------
-# Ideally we should use `AC_PROVIDE($1)', but unfortunately many third
-# party macros know that we use `AC_PROVIDE_$1' and they depend on it.
+# Ideally we should just use `m4_provide($1)', but unfortunately many
+# third party macros know that we use `AC_PROVIDE_$1' and they depend
+# on it.
 m4_define([AC_PROVIDE],
-[m4_define([AC_PROVIDE_$1])])
+[m4_define([AC_PROVIDE_$1])m4_provide([$1])])
 
 
 # AC_PROVIDE_IFELSE(MACRO-NAME, IF-PROVIDED, IF-NOT-PROVIDED)
diff --git a/lib/autoconf/general.m4 b/lib/autoconf/general.m4
index 7ae5b45d6..eadea64b2 100644
--- a/lib/autoconf/general.m4
+++ b/lib/autoconf/general.m4
@@ -264,10 +264,11 @@ m4_define([AC_EXPAND_ONCE],
 
 # AC_PROVIDE(MACRO-NAME)
 # ----------------------
-# Ideally we should use `AC_PROVIDE($1)', but unfortunately many third
-# party macros know that we use `AC_PROVIDE_$1' and they depend on it.
+# Ideally we should just use `m4_provide($1)', but unfortunately many
+# third party macros know that we use `AC_PROVIDE_$1' and they depend
+# on it.
 m4_define([AC_PROVIDE],
-[m4_define([AC_PROVIDE_$1])])
+[m4_define([AC_PROVIDE_$1])m4_provide([$1])])
 
 
 # AC_PROVIDE_IFELSE(MACRO-NAME, IF-PROVIDED, IF-NOT-PROVIDED)
diff --git a/tests/base.at b/tests/base.at
index b0211558e..86c7eff4c 100644
--- a/tests/base.at
+++ b/tests/base.at
@@ -7,13 +7,13 @@ Autoconf base layer.
 EOF
 
 
-## ------------ ##
-## AC_REQUIRE.  ##
-## ------------ ##
+## ------------------------------- ##
+## AC_REQUIRE: topological sort..  ##
+## ------------------------------- ##
 
 # Check that dependencies are always properly honored.
 
-AT_SETUP(AC_REQUIRE: topological sort)
+AT_SETUP([AC_REQUIRE: topological sort])
 
 AT_DATA(configure.in,
 [[define([REQUIRE_AND_CHECK],
@@ -54,7 +54,7 @@ AT_CLEANUP(configure)
 ## AC_REQUIRE and AC_DEFUN_ONCE: Require, expand.  ##
 ## ----------------------------------------------- ##
 
-AT_SETUP(AC_REQUIRE & AC_DEFUN_ONCE: Require, expand)
+AT_SETUP([AC_REQUIRE & AC_DEFUN_ONCE: Require, expand])
 
 AT_DATA([configure.in],
 [[AC_DEFUN([TEST],
@@ -97,7 +97,7 @@ AT_CLEANUP(configure)
 ## AC_REQUIRE and AC_DEFUN_ONCE: Expand, require.  ##
 ## ----------------------------------------------- ##
 
-AT_SETUP(AC_REQUIRE & AC_DEFUN_ONCE: Expand, require)
+AT_SETUP([AC_REQUIRE & AC_DEFUN_ONCE: Expand, require])
 
 AT_DATA([configure.in],
 [[AC_DEFUN([TEST],
@@ -122,6 +122,7 @@ case $multi_test:$single_test in
   ..:. ) AS_EXIT(0);;
   ..:* ) AC_MSG_ERROR([DEFUN_ONCE is broken]);;
   *:.  ) AC_MSG_ERROR([DEFUN is broken (Wow, congrats!)]);;
+     * ) AC_MSG_ERROR([received `$multi_test:$single_test']);;
 esac
 ]])
 
@@ -132,3 +133,34 @@ AT_CHECK([./configure], 0)
 
 
 AT_CLEANUP(configure)
+
+
+
+## ------------------------- ##
+## AC_REQUIRE & AC_PROVIDE.  ##
+## ------------------------- ##
+
+AT_SETUP([AC_REQUIRE & AC_PROVIDE])
+
+AT_DATA([configure.in],
+[[AC_DEFUN([TEST],
+[AC_REQUIRE([INNER_TEST])])
+
+AC_DEFUN([INNER_TEST],
+[inner_test=".$inner_test"])
+
+AC_PLAIN_SCRIPT
+AC_PROVIDE([INNER_TEST])
+TEST
+
+case $inner_test in
+  "" ) AS_EXIT(0);;
+  *  ) AC_MSG_ERROR([received `$inner_test']);;
+esac
+]])
+
+AT_CHECK([autoconf --autoconf-dir .. -l $at_srcdir], 0)
+AT_CHECK([./configure], 0)
+
+
+AT_CLEANUP(configure)