Stefano Lattarini <stefano.lattar...@gmail.com> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to