Skip to content

Commit

Permalink
build: improve GIT_CONF_SUBST signature
Browse files Browse the repository at this point in the history
Now, in configure.ac, a call like:

    GIT_CONF_SUBST([FOO])

will be considered equivalent to:

    GIT_CONF_SUBST([FOO], [$FOO])

This is mostly a preparatory refactoring in view of future changes.
No semantic change to the generated configure or config.mak.auto is
intended.

Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Stefano Lattarini authored and Junio C Hamano committed Sep 11, 2012
1 parent 682ce8b commit 3a34c2b
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,9 @@
# ------------------------
# Cause the line "VAR=VAL" to be eventually appended to ${config_file}.
AC_DEFUN([GIT_CONF_SUBST],
[AC_REQUIRE([GIT_CONF_SUBST_INIT])
config_appended_defs="$config_appended_defs${newline}$1=$2"])
[AC_REQUIRE([GIT_CONF_SUBST_INIT])
config_appended_defs="$config_appended_defs${newline}dnl
$1=m4_if([$#],[1],[${$1}],[$2])"])

# GIT_CONF_SUBST_INIT
# -------------------
Expand Down Expand Up @@ -179,7 +180,7 @@ AC_ARG_WITH([lib],
else
lib=$withval
AC_MSG_NOTICE([Setting lib to '$lib'])
GIT_CONF_SUBST([lib], [$withval])
GIT_CONF_SUBST([lib])
fi])

if test -z "$lib"; then
Expand Down Expand Up @@ -215,7 +216,7 @@ AC_ARG_ENABLE([jsmin],
[
JSMIN=$enableval;
AC_MSG_NOTICE([Setting JSMIN to '$JSMIN' to enable JavaScript minifying])
GIT_CONF_SUBST([JSMIN], [$enableval])
GIT_CONF_SUBST([JSMIN])
])

# Define option to enable CSS minification
Expand All @@ -225,7 +226,7 @@ AC_ARG_ENABLE([cssmin],
[
CSSMIN=$enableval;
AC_MSG_NOTICE([Setting CSSMIN to '$CSSMIN' to enable CSS minifying])
GIT_CONF_SUBST([CSSMIN], [$enableval])
GIT_CONF_SUBST([CSSMIN])
])

## Site configuration (override autodetection)
Expand Down Expand Up @@ -265,8 +266,8 @@ AS_HELP_STRING([], [ARG can be also prefix for libpcre library and hea
else
USE_LIBPCRE=YesPlease
LIBPCREDIR=$withval
AC_MSG_NOTICE([Setting LIBPCREDIR to $withval])
GIT_CONF_SUBST([LIBPCREDIR], [$withval])
AC_MSG_NOTICE([Setting LIBPCREDIR to $LIBPCREDIR])
GIT_CONF_SUBST([LIBPCREDIR])
fi)
#
# Define NO_CURL if you do not have curl installed. git-http-pull and
Expand Down

0 comments on commit 3a34c2b

Please sign in to comment.