diff mbox

weston: check for both EGL and GLES

Message ID 1460122957-3164-1-git-send-email-gustavo@zacarias.com.ar
State Accepted
Commit 1e14605ba859532d65bc598f9c66561528c6fd84
Headers show

Commit Message

Gustavo Zacarias April 8, 2016, 1:42 p.m. UTC
The conditional is wrong, it was check for BR2_PACKAGE_MESA3D_OPENGL_ES
(correct, means GLESv2 support) and BR2_PACKAGE_MESA3D_OPENGL_GLES
(incorrect, there's no such symbol).
So now check for BR2_PACKAGE_MESA3D_OPENGL_EGL (EGL support) plus
BR2_PACKAGE_MESA3D_OPENGL_ES (GLESv2 support), both are required.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/weston/weston.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Peter Korsgaard April 8, 2016, 8:11 p.m. UTC | #1
>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 > The conditional is wrong, it was check for BR2_PACKAGE_MESA3D_OPENGL_ES
 > (correct, means GLESv2 support) and BR2_PACKAGE_MESA3D_OPENGL_GLES
 > (incorrect, there's no such symbol).
 > So now check for BR2_PACKAGE_MESA3D_OPENGL_EGL (EGL support) plus
 > BR2_PACKAGE_MESA3D_OPENGL_ES (GLESv2 support), both are required.

 > Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

Committed, thanks.
diff mbox

Patch

diff --git a/package/weston/weston.mk b/package/weston/weston.mk
index 7706599..1f31097 100644
--- a/package/weston/weston.mk
+++ b/package/weston/weston.mk
@@ -48,7 +48,7 @@  WESTON_CONF_OPTS += --disable-weston-launch
 endif
 
 # Needs wayland-egl, which normally only mesa provides
-ifeq ($(BR2_PACKAGE_MESA3D_OPENGL_ES)$(BR2_PACKAGE_MESA3D_OPENGL_GLES),yy)
+ifeq ($(BR2_PACKAGE_MESA3D_OPENGL_EGL)$(BR2_PACKAGE_MESA3D_OPENGL_ES),yy)
 WESTON_CONF_OPTS += --enable-egl
 WESTON_DEPENDENCIES += libegl
 else