Re: [meta-qt5][master][jethro][PATCH] qtquick1: fix qtwebkit support

[Alexandre Belloni <alexandre.belloni@free-electrons.com>]

On 07/03/2016 at 13:13:24 +0100, Martin Jansa wrote :
> On Thu, Mar 03, 2016 at 09:52:09AM +0100, Alexandre Belloni wrote:
> > 40b7a93a3c6cf05136c606824c1cea848b75bcb6 ("qtquick1: allow to disable
> > qtwebkit support") allowed to disable qtwebkit but actually disabled it in
> > every cases.
> > 
> > Use the same mechanism as qttools to implement the feature.
> > 
> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> > ---
> >  recipes-qt/qt5/qtquick1_git.bb | 8 +++++---
> >  1 file changed, 5 insertions(+), 3 deletions(-)
> > 
> > diff --git a/recipes-qt/qt5/qtquick1_git.bb b/recipes-qt/qt5/qtquick1_git.bb
> > index e0a264d4fa2d..cb5e395b9d67 100644
> > --- a/recipes-qt/qt5/qtquick1_git.bb
> > +++ b/recipes-qt/qt5/qtquick1_git.bb
> > @@ -16,11 +16,13 @@ LIC_FILES_CHKSUM = " \
> >  DEPENDS += "qtscript qtsvg qtxmlpatterns"
> >  # qttools
> >  
> > -PACKAGECONFIG ??= "webkit"
> > -PACKAGECONFIG[webkit] = "CONFIG+=enable-webkit,CONFIG-=enable-webkit,qtwebkit"
> > +PACKAGECONFIG ??= "qtwebkit"
> > +PACKAGECONFIG[qtwebkit] = ",,qtwebkit"
> 
> Is there reason for renaming the packageconfig option?
> 

I wanted to align with the qttools option.

> Where did you see this issue? Weren't you testing master-next with:
> http://patchwork.openembedded.org/patch/116717/
> without corresponding oe-core change?
> 

It was on jethro. I'll try a NODISTRO build to see whether this is
caused by poky.


-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com