On 11/29/2015 05:52 AM, Thomas Orgis wrote:
Am Fri, 27 Nov 2015 22:04:25 +0100
schrieb Gilles Chanteperdrix<gilles.chanteperd...@xenomai.org>:
I think I posted a mail about the same bug for libva-intel-driver
1.6.0 in July:
http://lists.freedesktop.org/archives/libva/2015-July/003413.html
Indeed, you report the exact same findings and a very similar patch.
Now, I would not mind if _any_ of the proposed variants (including the
Mesa approach postet by Emil) gets tested once without and _with_
wayland and included in libva upstream. I'm not particulary attached to
my variant of the patch;-)
Dear libva team: What is prohibiting inclusion of a fix for bug 79478?
Hi, Thomas
Thanks for your patch. Your effort is appreciated.
I did one minor change based on your patch. If the wayland is not
used or found, it will skip the va/wayland/Makefile.
Thanks
Yakui
Alrighty then,
Thomas
_______________________________________________
Libva mailing list
Libva@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libva
From 0e170fe45203f909a089b715bc34611b2a4af39a Mon Sep 17 00:00:00 2001
From: Zhao Yakui <yakui.z...@intel.com>
Date: Mon, 30 Nov 2015 01:45:18 -0500
Subject: [PATCH] Skip the check related with wayland if wayland is not used or
found
https://bugs.freedesktop.org/show_bug.cgi?id=79478
Signed-off-by: Thomas Orgis <obu...@sourcemage.org>
Signed-off-by: Zhao Yakui <yakui.z...@intel.com>
---
configure.ac | 20 +++++++++++++-------
1 file changed, 13 insertions(+), 7 deletions(-)
diff --git a/configure.ac b/configure.ac
index 88a7ef3..9d2c3a3 100644
--- a/configure.ac
+++ b/configure.ac
@@ -273,15 +273,16 @@ if test "$enable_wayland" = "yes"; then
PKG_CHECK_MODULES([WAYLAND], [wayland-client >= wayland_api_version],
[USE_WAYLAND="yes"], [:])
if test "$USE_WAYLAND" = "yes"; then
+ m4_ifdef([WAYLAND_SCANNER_RULES],
+ [WAYLAND_SCANNER_RULES(['$(top_builddir)/va/wayland/protocol'])],
+ [wayland_scanner_rules=""; AC_SUBST(wayland_scanner_rules)])
+
AC_DEFINE([HAVE_VA_WAYLAND], [1],
[Defined to 1 if VA/Wayland API is built])
fi
fi
-AM_CONDITIONAL(USE_WAYLAND, test "$USE_WAYLAND" = "yes")
-m4_ifdef([WAYLAND_SCANNER_RULES],
- [WAYLAND_SCANNER_RULES(['$(top_builddir)/va/wayland/protocol'])],
- [wayland_scanner_rules=""; AC_SUBST(wayland_scanner_rules)])
+AM_CONDITIONAL(USE_WAYLAND, test "$USE_WAYLAND" = "yes")
pkgconfigdir=${libdir}/pkgconfig
AC_SUBST(pkgconfigdir)
@@ -309,8 +310,6 @@ debian.upstream/libva-drm.install.in
debian.upstream/libva-egl.install.in
debian.upstream/libva-glx$LIBVA_MAJOR_VERSION.install:\
debian.upstream/libva-glx.install.in
- debian.upstream/libva-wayland$LIBVA_MAJOR_VERSION.install:\
-debian.upstream/libva-wayland.install.in
debian.upstream/libva-x11-$LIBVA_MAJOR_VERSION.install:\
debian.upstream/libva-x11.install.in
doc/Makefile
@@ -335,10 +334,17 @@ debian.upstream/libva-x11.install.in
va/egl/Makefile
va/glx/Makefile
va/va_version.h
+ va/x11/Makefile
+])
+
+if test "$USE_WAYLAND" = "yes"; then
+AC_OUTPUT([
+ debian.upstream/libva-wayland$LIBVA_MAJOR_VERSION.install:\
+debian.upstream/libva-wayland.install.in
va/wayland/Makefile
va/wayland/protocol/Makefile
- va/x11/Makefile
])
+fi
# Print a small summary
BACKENDS=""
--
1.8.2.1
_______________________________________________
Libva mailing list
Libva@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libva
