Re: [Mesa-dev] [PATCH 26/28] configure: error out if building XVMC w/o supported platform

2016-12-13 Thread Nayan Deshmukh
Half of this patch went in 25, with that fixed both are Reviewed-by:
Nayan Deshmukh 

On Fri, Dec 9, 2016 at 1:30 AM, Emil Velikov  wrote:
> From: Emil Velikov 
>
> Signed-off-by: Emil Velikov 
> ---
>  configure.ac | 5 -
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index a4327ad..606ea98 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2123,7 +2123,7 @@ dnl
>  dnl Gallium G3DVL configuration
>  dnl
>  if test -n "$with_gallium_drivers" -a "x$with_gallium_drivers" != xswrast; 
> then
> -if test "x$enable_xvmc" = xauto; then
> +if test "x$enable_xvmc" = xauto -a "x$have_xvmc_platform" = xyes; then
> PKG_CHECK_EXISTS([xvmc >= $XVMC_REQUIRED], [enable_xvmc=yes], 
> [enable_xvmc=no])
>  fi
>
> @@ -2159,6 +2159,9 @@ fi
>  AM_CONDITIONAL(NEED_GALLIUM_VL_WINSYS, test "x$need_gallium_vl_winsys" = 
> xyes)
>
>  if test "x$enable_xvmc" = xyes; then
> +if test "x$have_xvmc_platform" != xyes; then
> +AC_MSG_ERROR([XVMC requires the x11 platforms])
> +fi
>  PKG_CHECK_MODULES([XVMC], [xvmc >= $XVMC_REQUIRED])
>  gallium_st="$gallium_st xvmc"
>  fi
> --
> 2.10.2
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [PATCH 26/28] configure: error out if building XVMC w/o supported platform

2016-12-08 Thread Emil Velikov
From: Emil Velikov 

Signed-off-by: Emil Velikov 
---
 configure.ac | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index a4327ad..606ea98 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2123,7 +2123,7 @@ dnl
 dnl Gallium G3DVL configuration
 dnl
 if test -n "$with_gallium_drivers" -a "x$with_gallium_drivers" != xswrast; then
-if test "x$enable_xvmc" = xauto; then
+if test "x$enable_xvmc" = xauto -a "x$have_xvmc_platform" = xyes; then
PKG_CHECK_EXISTS([xvmc >= $XVMC_REQUIRED], [enable_xvmc=yes], 
[enable_xvmc=no])
 fi
 
@@ -2159,6 +2159,9 @@ fi
 AM_CONDITIONAL(NEED_GALLIUM_VL_WINSYS, test "x$need_gallium_vl_winsys" = xyes)
 
 if test "x$enable_xvmc" = xyes; then
+if test "x$have_xvmc_platform" != xyes; then
+AC_MSG_ERROR([XVMC requires the x11 platforms])
+fi
 PKG_CHECK_MODULES([XVMC], [xvmc >= $XVMC_REQUIRED])
 gallium_st="$gallium_st xvmc"
 fi
-- 
2.10.2

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev