[oe] [meta-browser][PATCH] chromium-wayland: add wayland to REQUIRED_DISTRO_FEATURES
Martin Jansa
martin.jansa at gmail.com
Thu Oct 13 18:32:48 UTC 2016
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.Jansa at 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.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 163 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20161013/92d42358/attachment-0002.sig>
More information about the Openembedded-devel
mailing list