Bruno Haible <bruno@xxxxxxxxx> wrote: > James Youngman wrote: >> The following patch fixes the problem. I'm not sure if it is >> the *right* fix for the problem, since I have only been working with >> Autoconf/Automake macros for ten years, and so I don't understand it >> well enough yet. >> >> 2007-01-22 James Youngman <jay@xxxxxxx> >> >> * m4/gnulib-common.m4 (AC_PROG_MKDIR_P): If AC_PROG_MKDIR_P >> is already provided, call AC_SUBST on MKDIR_P anyway, since >> the version of AC_PROG_MKDIR_P in Autoconf-2.61 (at least) >> doesn't do that. > > Yes, this appears right. I'll give it more thorough testing tomorrow, > but for now I've applied your patch (with different comments only). Thanks. I thought so too, at first, but then saw this comment in autoconf's lib/autoconf/programs.m4: dnl Do special magic for MKDIR_P instead of AC_SUBST, to get dnl relative names right. AC_MSG_RESULT([$MKDIR_P]) The only case I can imagine is when MKDIR_P is specified as e.g., MKDIR_P='../build-aux/install-sh -d' in one place, and later (in another Makefile) as MKDIR_P='somewhere/else/install-sh -d' in another. I wonder which use case made it so using AC_SUBST wasn't an option. Here's a related ChangeLog entry from autoconf (rev. 1.53 of lib/autoconf/programs.m4): 2006-05-13 Paul Eggert <eggert@xxxxxxxxxxx> * doc/autoconf.texi (Particular Programs): AC_PROG_MKDIR_P now sets MKDIR_P, not mkdir_p, to avoid collisions with Automake. Warn about obsolete install-sh files. Remove stray sentence fragment and fix cross reference. * lib/autoconf/programs.m4 (AC_PROG_INSTALL): Don't insist on install -d; this undoes the 2006-05-10 change. (MKDIR_P): Mark with AN_MAKEVAR. (AC_PROG_MKDIR_P): Fall back on $ac_install_sh, not $INSTALL, so that we don't require $INSTALL to be thread-safe. Move comments out of generated code. Require AC_CONFIG_AUX_DIR_DEFAULT instead of AC_PROG_INSTALL. Output a message saying that we're checking mkdir -p. Set MKDIR_P rather than mkdir_p. Do special magic for MKDIR_P instead of AC_SUBST. * lib/autoconf/status.m4 (_AC_OUTPUT_FILE, _AC_OUTPUT_CONFIG_STATUS): Special magic for MKDIR_P, too. * lib/m4sugar/m4sh.m4 (AS_MKDIR_P): Remove comment that defeated a dnl. * tests/local.at (AT_CHECK_ENV): mkdir_p -> MKDIR_P. _______________________________________________ Autoconf mailing list Autoconf@xxxxxxx http://lists.gnu.org/mailman/listinfo/autoconf