[meta-freescale] [PATCH 1/2] gstreamer1.0-plugins-imx: select Wayland EGL platform if appropriate

[Otavio Salvador]

On Wed, Jun 25, 2014 at 6:33 PM, Carlos Rafael Giani
<dv at pseudoterminal.org> wrote:
> Signed-off-by: Carlos Rafael Giani <dv at pseudoterminal.org>
> ---
>  .../gstreamer/gstreamer1.0-plugins-imx_0.9.6.bb          | 16 +++++++++++++++-
>  1 file changed, 15 insertions(+), 1 deletion(-)
>
> diff --git a/recipes-multimedia/gstreamer/gstreamer1.0-plugins-imx_0.9.6.bb b/recipes-multimedia/gstreamer/gstreamer1.0-plugins-imx_0.9.6.bb
> index 5074a2c..4504563 100644
> --- a/recipes-multimedia/gstreamer/gstreamer1.0-plugins-imx_0.9.6.bb
> +++ b/recipes-multimedia/gstreamer/gstreamer1.0-plugins-imx_0.9.6.bb
> @@ -14,7 +14,21 @@ S = "${WORKDIR}/git"
>  inherit waf
>
>  # configure the eglvivsink element to use the appropiate EGL platform code
> -EGLVIVSINK_PLATFORM = "${@base_contains('DISTRO_FEATURES', 'x11', 'x11', 'fb', d)}"
> +# X11 if x11 is present in DISTRO_FEATURES
> +# Wayland if x11 is not present in DISTRO_FEATURES, but wayland is
> +# Framebuffer otherwise
> +EGLVIVSINK_PLATFORM = "fb"
> +
> +python() {
> +    eglvivsink_platform = 'fb'
> +    if base_contains('DISTRO_FEATURES', 'x11', True, False, d):
> +        eglvivsink_platform = 'x11'
> +    elif base_contains('DISTRO_FEATURES', 'wayland', True, False, d):
> +        eglvivsink_platform = 'wayland'
> +
> +    d.setVar('EGLVIVSINK_PLATFORM', eglvivsink_platform)
> +}

I agree with the goal of the patch and it does makes sense to do it
automatically. The use of python anonymous should be avoided when
possible though. Please look at the cairo[1] bbappend how we handle
the three cases using the inline python code which is more commonly
used for this kind of thing.

Can you rework it?

-- 
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