[ moving from autoconf@ to -patches ]
* Brian Gough wrote on Mon, Feb 07, 2011 at 01:45:04PM CET:
> Maybe it would help to publicise "-C" in the output so people
> are more aware of it. e.g.
>
> $ ./configure
> configure: not caching test results (use -C to enable)
> checking for a BSD-compatible install... /usr/bin/install -c
> ...
>
> The message could be suppressed if config.status does not exist, so
> it's only shown if someone is configuring more than once.
This would implement that suggestion. I'm not totally sold on this
(and I'm making it a _NOTICE not a _WARNING so it is hidden with -q),
so I'd appreciate input from the other developers if we want this.
Thanks,
Ralf
Hint at -C option when running configure repeatedly.
* lib/autoconf/general.m4 (AC_CACHE_LOAD): Note disabled cache
when config.status is present.
* tests/base.at (AC_CACHE_CHECK): Extend test.
Suggestion from Brian Gough.
diff --git a/lib/autoconf/general.m4 b/lib/autoconf/general.m4
index dd20e71..337aba7 100644
--- a/lib/autoconf/general.m4
+++ b/lib/autoconf/general.m4
@@ -1915,6 +1915,8 @@ m4_define([AC_CACHE_LOAD],
[[\\/]]* | ?:[[\\/]]* ) . "$cache_file";;
*) . "./$cache_file";;
esac
+ elif test -x ./config.status; then
+ AC_MSG_NOTICE([not caching test results (use -C to enable)])
fi
else
AC_MSG_NOTICE([creating cache $cache_file])
diff --git a/tests/base.at b/tests/base.at
index 4b24366..7fac91e 100644
--- a/tests/base.at
+++ b/tests/base.at
@@ -466,6 +466,13 @@ AT_CHECK_CONFIGURE([-C])
# should be updated.
AT_CHECK([test -s cache || test ! -h config.cache])
+# Hint at -C if not running configure for the first time.
+rm -f cache config.cache config.status
+AT_CHECK_CONFIGURE([], [], [stdout])
+AT_CHECK([grep 'not caching test results' stdout], [1])
+AT_CHECK_CONFIGURE([], [], [stdout])
+AT_CHECK([grep 'not caching test results' stdout], [0], [ignore])
+
# config.site can specify a site-wide cache, accumulating information.
# Also test that we don't run afoul of sourcing a file with leading -.
AT_DATA([-config.site],
