Hi Andreas On Tue, Aug 28, 2018 at 10:21 AM Andreas Müller <[email protected]> wrote: > > On Tue, Aug 28, 2018 at 8:43 AM, Andreas Müller <[email protected]> > wrote: > > On Tue, Aug 28, 2018 at 12:55 AM, Khem Raj <[email protected]> wrote: > >> On Mon, Aug 27, 2018 at 2:15 PM Andreas Müller <[email protected]> > >> wrote: > >>> > >>> On Mon, Aug 20, 2018 at 8:59 PM, Khem Raj <[email protected]> wrote: > >>> > Signed-off-by: Khem Raj <[email protected]> > >>> > --- > >>> > meta/recipes-graphics/cairo/cairo.inc | 3 ++- > >>> > meta/recipes-graphics/mesa/mesa.inc | 3 +++ > >>> > 2 files changed, 5 insertions(+), 1 deletion(-) > >>> > > >>> > diff --git a/meta/recipes-graphics/cairo/cairo.inc > >>> > b/meta/recipes-graphics/cairo/cairo.inc > >>> > index 20e0d2c92a..7347f223ff 100644 > >>> > --- a/meta/recipes-graphics/cairo/cairo.inc > >>> > +++ b/meta/recipes-graphics/cairo/cairo.inc > >>> > @@ -22,7 +22,8 @@ X11DEPENDS = "virtual/libx11 libsm libxrender libxext" > >>> > DEPENDS = "libpng fontconfig pixman glib-2.0 zlib" > >>> > > >>> > PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 > >>> > xcb', '', d)} \ > >>> > - ${@bb.utils.filter('DISTRO_FEATURES', 'directfb', d)}" > >>> > + ${@bb.utils.filter('DISTRO_FEATURES', 'directfb', d)} \ > >>> > + ${@bb.utils.contains('DISTRO_FEATURES', 'x11 opengl', > >>> > 'opengl', '', d)}" > >>> > > >>> > PACKAGECONFIG[x11] = "--with-x=yes -enable-xlib,--with-x=no > >>> > --disable-xlib,${X11DEPENDS}" > >>> > PACKAGECONFIG[xcb] = "--enable-xcb,--disable-xcb,libxcb" > >>> > diff --git a/meta/recipes-graphics/mesa/mesa.inc > >>> > b/meta/recipes-graphics/mesa/mesa.inc > >>> > index 5afd0db4b7..dd626d9f00 100644 > >>> ^ I think this came in accidentaly and it breaks builds when adding > >>> packageconfig glesv2: > >>> > >>> | configure: error: use either --enable-gl=yes or --enable-glesv2=yes. > >>> Not both at the same time. > >> > >> do you remove opengl from DISTRO_FEATURES when enabling glesv2 ? > >> try that out. > >> > >>> > > Yes of course I can. But this patch is wrong - we don't have gles as > > distro feature - and it is not even mentioned in commit message why > > this is done. > > > I am tempted to send a revert for the cairo part: > > * Cross: It changes defaults valid for long time > * Native: As far as I can remember it causes trouble for cairo-native > * It went through without mentioning
opengl as a DISTRO_FEATURE should tie into packageconfigs, the situation before this patch was not ideal, people got to work with it. as seen in the example you cited, we can't mix opengl and gles support in packages. other layers have solved your kind of usecase like below https://github.com/WebPlatformForEmbedded/meta-wpe/blob/master/recipes-graphics/cairo/cairo_%.bbappend > > Andreas -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
