[v2] qt5wayland: compositor depends on qt5base opengl

Message ID 1518540433-13500-1-git-send-email-joshua.henderson@microchip.com
State New
Headers show
Series
  • [v2] qt5wayland: compositor depends on qt5base opengl
Related show

Commit Message

Joshua Henderson Feb. 13, 2018, 4:47 p.m.
When building Qt 5.6.3, the qt5wayland compositor depends on Qt OpenGL support.
The compositor API is marked experimental prior to Qt 5.8.  In the latest
version of Qt this dependendency on opengl is removed.

This also removes the qt5wayland dependency on GL support, because this is tied
to the compositor API through the BR2_PACKAGE_QT5BASE_OPENGL dependency.

Fixes:
      http://autobuild.buildroot.net/results/289d9b73d99e29d61c2df9ce228fed3968d89f5c

Cc: Peter Seiderer <ps.report@gmx.net>
Cc: Julien Corjon <corjon.j@ecagroup.com>
Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>

---

v2:
- Due to the compositor API being experimental in Qt 5.6.3, it does not make
  sense to address this dependency on OpenGL upstream in this version of Qt.
- Remove explicit dependency on GL support.
- Note that the compositor API is marked as stable starting with Qt 5.8 in help.
- Update commit message.
---
 package/qt5/qt5wayland/Config.in | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

Patch

diff --git a/package/qt5/qt5wayland/Config.in b/package/qt5/qt5wayland/Config.in
index 0182d3c..29755cf 100644
--- a/package/qt5/qt5wayland/Config.in
+++ b/package/qt5/qt5wayland/Config.in
@@ -1,10 +1,5 @@ 
-comment "qt5wayland needs an OpenGL-capable backend"
-	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
-	depends on !BR2_PACKAGE_QT5_GL_AVAILABLE
-
 config BR2_PACKAGE_QT5WAYLAND
 	bool "qt5wayland"
-	depends on BR2_PACKAGE_QT5_GL_AVAILABLE
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE # qt5declarative
 	select BR2_PACKAGE_QT5DECLARATIVE
 	select BR2_PACKAGE_WAYLAND
@@ -18,10 +13,13 @@  config BR2_PACKAGE_QT5WAYLAND
 
 if BR2_PACKAGE_QT5WAYLAND
 
+comment "qt5wayland compositor needs Qt OpenGL support"
+	depends on !BR2_PACKAGE_QT5BASE_OPENGL && !BR2_PACKAGE_QT5_VERSION_LATEST
+
 config BR2_PACKAGE_QT5WAYLAND_COMPOSITOR
 	bool "Enable compositor (experimental)"
+	depends on BR2_PACKAGE_QT5BASE_OPENGL || BR2_PACKAGE_QT5_VERSION_LATEST
 	help
-	  The compositor API is still experimental, and not built by
-	  default.
+	  The compositor API is stable as of Qt 5.8.
 
 endif