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 <kasal@xxxxxx> * 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 Autoconf@xxxxxxx http://lists.gnu.org/mailman/listinfo/autoconf
