Stefano Lattarini <stefano.lattarini@xxxxxxxxx> writes: > diff --git a/configure.ac b/configure.ac > index 789926f..9472f6b 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -29,13 +29,12 @@ AC_DEFUN([GIT_ARG_SET_PATH], > # --without-PROGRAM is used. > AC_DEFUN([GIT_CONF_APPEND_PATH], > [m4_pushdef([GIT_UC_PROGRAM], m4_toupper([$1]))dnl > if test "$withval" = "no"; then > if test -n "$2"; then > GIT_UC_PROGRAM[]_PATH=$withval > + AC_MSG_NOTICE([Disabling use of GIT_UC_PROGRAM]) > + GIT_CONF_SUBST([NO_]GIT_UC_PROGRAM, [YesPlease]) > + GIT_CONF_SUBST(GIT_UC_PROGRAM[_PATH], []) Nice. The inconsistency between the existing one that does not quote the string that is not substituted, i.e. GIT_UC_PROGRAM[]_PATH=$withval and the new one that quotes _PATH, i.e. GIT_CONF_SUBST(GIT_UC_PROGRAM[_PATH], []) looks somewhat strange, though. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html