On Thu, Oct 13, 2016 at 04:24:32PM +0200, Martin Jansa wrote:
> * otherwise chromium-wayland recipe might build with CHROMIUM_ENABLE_WAYLAND 
> 0 and then
>   autodetect couple of x11 libraries
>   chromium-wayland-48.0.2548.0: chromium-wayland rdepends on atk, but it 
> isn't a build dependency, missing atk in DEPENDS or PACKAGECONFIG? 
> [build-deps]
>   chromium-wayland-48.0.2548.0: chromium-wayland rdepends on gdk-pixbuf, but 
> it isn't a build dependency, missing gdk-pixbuf in DEPENDS or PACKAGECONFIG? 
> [build-deps]
>   chromium-wayland-48.0.2548.0: chromium-wayland rdepends on gtk+, but it 
> isn't a build dependency, missing gtk+ in DEPENDS or PACKAGECONFIG? 
> [build-deps]
>   chromium-wayland-48.0.2548.0: chromium-wayland rdepends on libxcomposite, 
> but it isn't a build dependency, missing libxcomposite in DEPENDS or 
> PACKAGECONFIG? [build-deps]
>   chromium-wayland-48.0.2548.0: chromium-wayland rdepends on libxcursor, but 
> it isn't a build dependency, missing libxcursor in DEPENDS or PACKAGECONFIG? 
> [build-deps]
>   chromium-wayland-48.0.2548.0: chromium-wayland rdepends on libxscrnsaver, 
> but it isn't a build dependency, missing libxscrnsaver in DEPENDS or 
> PACKAGECONFIG? [build-deps]
> 
> * reported a while ago in:
>   
> http://lists.openembedded.org/pipermail/openembedded-devel/2016-September/109131.html
> 
> Signed-off-by: Martin Jansa <martin.ja...@gmail.com>
> ---
>  recipes-browser/chromium/chromium-wayland_48.0.2548.0.bb | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/recipes-browser/chromium/chromium-wayland_48.0.2548.0.bb 
> b/recipes-browser/chromium/chromium-wayland_48.0.2548.0.bb
> index f7ad2ce..bad3d71 100644
> --- a/recipes-browser/chromium/chromium-wayland_48.0.2548.0.bb
> +++ b/recipes-browser/chromium/chromium-wayland_48.0.2548.0.bb
> @@ -2,6 +2,9 @@ include chromium-browser.inc
>  
>  DEPENDS += "wayland libxkbcommon"
>  
> +inherit distro_features_check
> +REQUIRED_DISTRO_FEATURES = "wayland"

This isn't enough, because when x11 is also in DISTRO_FEATURES it will
disable wayland support (so we should skip this recipe).

# $CHROMIUM_ENABLE_WAYLAND
#   set
#   
/OE/build/oe-core/meta-browser/recipes-browser/chromium/chromium-browser.inc:109
#     [_defaultval] "${@bb.utils.contains('DISTRO_FEATURES', 'x11', '0',
#     bb.utils.contains('DISTRO_FEATURES', 'wayland', '1',
#     '0', d),d)}"
CHROMIUM_ENABLE_WAYLAND="0"

> +
>  SRC_URI += "\
>          file://chromium-wayland/add_missing_stat_h_include.patch \
>          file://0003-Remove-hard-coded-values-for-CC-and-CXX.patch \
> -- 
> 2.10.1
> 

-- 
Martin 'JaMa' Jansa     jabber: martin.ja...@gmail.com

Attachment: signature.asc
Description: Digital signature

-- 
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Reply via email to