Hello,
On Tue, Sep 12, 2006 at 01:42:41PM -0400, Stefan Seefeld wrote:
> [...] And yes, bailing out in configure's execution if the directory
> isn't present, but then never touch the directory again, isn't contributing
> to clarify anything either. :-)
I agree with this, configure should not check for the existence of
the directory. (It would be a job for distcheck.)
Moreover, I do not think it makes sense to specify an absolute
directory there.
Patch attached, OK to commit?
Stepan
2006-09-12 Stepan Kasal <[EMAIL PROTECTED]>
* lib/autoconf/general.m4 (AC_CONFIG_AUX_DIR): Remove a
mistaken comment; the path has to be relative.
(AC_CONFIG_MACRO_DIR): Likewise; moreover, do not check for
the existence of the dir at configure time.
Index: lib/autoconf/general.m4
===================================================================
RCS file: /cvsroot/autoconf/autoconf/lib/autoconf/general.m4,v
retrieving revision 1.928
diff -u -r1.928 general.m4
--- lib/autoconf/general.m4 2 Sep 2006 05:37:08 -0000 1.928
+++ lib/autoconf/general.m4 12 Sep 2006 18:04:48 -0000
@@ -1565,7 +1565,6 @@
# ----------------------
# Find install-sh, config.sub, config.guess, and Cygnus configure
# in directory DIR. These are auxiliary files used in configuration.
-# DIR can be either absolute or relative to $srcdir.
AC_DEFUN([AC_CONFIG_AUX_DIR],
[AC_CONFIG_AUX_DIRS($1 "$srcdir"/$1)])
@@ -1627,15 +1626,7 @@
# AC_CONFIG_MACRO_DIR(DIR)
# ------------------------
# Declare directory containing additional macros for aclocal.
-# DIR can be either absolute or relative to $srcdir.
-AC_DEFUN([AC_CONFIG_MACRO_DIR],
-[case $1 in
- [[\\/]]* | ?:[[\\/]]* ) ac_macro_dir=$1 ;;
- *) ac_macro_dir=$srcdir/$1 ;;
-esac
-test -d "$ac_macro_dir" ||
- AC_MSG_ERROR([cannot find macro directory `$1'])
-])# AC_CONFIG_MACRO_DIR
+AC_DEFUN([AC_CONFIG_MACRO_DIR], [])
_______________________________________________
Autoconf mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/autoconf
