[meta-freescale] [meta-fsl-arm][PATCH 5/6] mesa-demos: Use wayland backend whenever it is availiable

[Otavio Salvador]

On Fri, Oct 16, 2015 at 1:18 PM, Neena Busireddy
<neena.busireddy at freescale.com> wrote:
> If Wayland exists in DISTRO features, wayland egl build configuration will be enabled.
>
> It is not mandatory for a GPU driver to support multiple backends (X11, wayland) at the same time.
>
> Signed-off-by: Neena Busireddy <neena.busireddy at freescale.com>
> ---
>  recipes-graphics/mesa/mesa-demos_%.bbappend |   12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/recipes-graphics/mesa/mesa-demos_%.bbappend b/recipes-graphics/mesa/mesa-demos_%.bbappend
> index 1f070a4..3bc87c8 100644
> --- a/recipes-graphics/mesa/mesa-demos_%.bbappend
> +++ b/recipes-graphics/mesa/mesa-demos_%.bbappend
> @@ -4,13 +4,15 @@ SRC_URI_append_mx6 = " file://Replace-glWindowPos2iARB-calls-with-glWindowPos2i.
>                      file://fix-clear-build-break.patch \
>                      file://Additional-eglSwapBuffer-calling-makes-wrong-throttl.patch \
>                      file://Add-OpenVG-demos-to-support-wayland.patch"
> -

Please don't remove this empty line.

>  PACKAGECONFIG_remove_mx6 = " \
> -    ${@bb.utils.contains('DISTRO_FEATURES', 'x11', '', \
> -         bb.utils.contains('DISTRO_FEATURES', 'wayland', 'glu', '', d), d)}"
> +    ${@bb.utils.contains('DISTRO_FEATURES', 'x11', \
> +        bb.utils.contains('DISTRO_FEATURES', 'wayland','x11 glu','',d), \
> +           bb.utils.contains('DISTRO_FEATURES', 'wayland','glu','',d),d)}"
> +
>  PACKAGECONFIG_remove_mx6sl = "gles1 gles2"

Ok.

>  PACKAGECONFIG_append = "\
> -    ${@bb.utils.contains('DISTRO_FEATURES', 'x11', '', \
> -       bb.utils.contains('DISTRO_FEATURES', 'wayland', 'wayland vg', '', d), d)} \
> +    ${@bb.utils.contains('DISTRO_FEATURES', 'x11', \
> +        bb.utils.contains('DISTRO_FEATURES', 'wayland','wayland vg','',d), \
> +             bb.utils.contains('DISTRO_FEATURES', 'wayland', 'wayland vg', '', d), d)} \
>  "

This one can be simplified.

PACKAGECONFIG_append_mx6 = " ${@bb.utils.contains('DISTRO_FEATURES',
'wayland', 'wayland vg', '', d)}"

Or am I missing anything?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750