[PATCH weston 09/11] configure.ac: Protect AC_PATH_PROG variables

Kristian Høgsberg hoegsberg at gmail.com
Mon Dec 16 17:05:43 PST 2013


On Sun, Dec 08, 2013 at 07:45:41PM +0100, Quentin Glidic wrote:
> From: Quentin Glidic <sardemff7+git at sardemff7.net>

The AC_ARG_VAR additions look useful, but again, I don't like the
superfluous quoting.

Kristian

> Signed-off-by: Quentin Glidic <sardemff7+git at sardemff7.net>
> ---
>  configure.ac | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 4f3e314..4e76a2c 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -516,13 +516,15 @@ if test "x$have_lcms" = "xyes"; then
>  fi
>  AM_CONDITIONAL(HAVE_LCMS, [test "x$have_lcms" = "xyes"])
>  
> +PKG_CHECK_MODULES(WAYLAND_SCANNER, [wayland-scanner])
> +AC_ARG_VAR([wayland_scanner],[The wayland-scanner executable])
>  AC_PATH_PROG([wayland_scanner], [wayland-scanner])
>  if test "x$wayland_scanner" = "x"; then
>  	AC_MSG_ERROR([wayland-scanner is needed to compile weston])
>  fi
>  
> -PKG_CHECK_MODULES(WAYLAND_SCANNER, wayland-scanner)
> -AC_PATH_PROG(XMLLINT, xmllint)
> +AC_ARG_VAR([XMLLINT],[The xmllint executable])
> +AC_PATH_PROG([XMLLINT], [xmllint])
>  AC_ARG_WITH(dtddir,
>  	    AS_HELP_STRING([--with-dtddir],
>  			   [Directory containing the Wayland
> -- 
> 1.8.4.3
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list