All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [meta-qt5][PATCH] qtwebkit: accept gstreamer 0.10 as an option
Date: Fri, 28 Feb 2014 02:13:01 +0100	[thread overview]
Message-ID: <20140228011301.GG26981@jama> (raw)
In-Reply-To: <530FDF58.8040505@opendreambox.org>

[-- Attachment #1: Type: text/plain, Size: 3127 bytes --]

On Fri, Feb 28, 2014 at 01:59:04AM +0100, Andreas Oberritter wrote:
> On 26.02.2014 22:05, Martin Jansa wrote:
> > On Wed, Feb 26, 2014 at 09:25:51PM +0100, Andreas Oberritter wrote:
> >> There's no configuration knob, though. QtWebKit will prefer
> >> gstreamer 1.0 if available.
> > 
> > Can you please add configuration knob first?
> > 
> > The problem is that without the knob it will be undeterministic, with
> > gstreamer010 in PACKAGECONFIG you can run 2nd build with gstreamer10
> > already staged for some other component and the resulting qtwebkit will
> > be different than in 1st build.
> 
> I'll see what I can do. I'm not a big fan of (Qt)WebKit's build system,
> so this might take a while.

You can use the same hack as I've used in qtmultimedia, it's not nice,
but works.

> FWIW, I noticed that qtlocation and qtsensors are optional build
> dependencies which need packageconfigs, too.

True, we're even disabling qtlocation in some our builds (because of
qt3d dependency), so having PACKAGECONFIG would be very appreciated.


> >> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> >> ---
> >>  recipes-qt/qt5/qtwebkit.inc | 6 +++++-
> >>  1 file changed, 5 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/recipes-qt/qt5/qtwebkit.inc b/recipes-qt/qt5/qtwebkit.inc
> >> index aea9dad..5e96f39 100644
> >> --- a/recipes-qt/qt5/qtwebkit.inc
> >> +++ b/recipes-qt/qt5/qtwebkit.inc
> >> @@ -5,7 +5,11 @@ LIC_FILES_CHKSUM = "file://Source/WebCore/rendering/RenderApplet.h;endline=22;md
> >>                      file://Source/WebKit/gtk/webkit/webkit.h;endline=21;md5=b4fbe9f4a944f1d071dba1d2c76b3351 \
> >>                      file://Source/JavaScriptCore/parser/Parser.h;endline=21;md5=bd69f72183a7af673863f057576e21ee"
> >>  
> >> -DEPENDS += "qtdeclarative qtlocation qtsensors ${ICU} ruby-native sqlite3 glib-2.0 gstreamer1.0 gstreamer1.0-plugins-base libxslt"
> >> +DEPENDS += "qtdeclarative qtlocation qtsensors ${ICU} ruby-native sqlite3 glib-2.0 libxslt"
> >> +
> >> +PACKAGECONFIG ??= "gstreamer10"
> >> +PACKAGECONFIG[gstreamer010] = ",,gstreamer gst-plugins-base"
> >> +PACKAGECONFIG[gstreamer10] = ",,gstreamer1.0 gstreamer1.0-plugins-base"
> >>  
> >>  # qtwebkit gets terribly big when linking with all debug info, disable by default
> >>  QTWEBKIT_DEBUG = "QMAKE_CFLAGS+=-g0 QMAKE_CXXFLAGS+=-g0"
> >> -- 
> >> 1.8.3.2
> >>
> >> _______________________________________________
> >> Openembedded-devel mailing list
> >> Openembedded-devel@lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> > 
> > 
> > 
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> > 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

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

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

  reply	other threads:[~2014-02-28  1:13 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-26 20:25 [meta-qt5][PATCH] qtwebkit: accept gstreamer 0.10 as an option Andreas Oberritter
2014-02-26 21:05 ` Martin Jansa
2014-02-28  0:59   ` Andreas Oberritter
2014-02-28  1:13     ` Martin Jansa [this message]
2014-02-28 16:59       ` Denys Dmytriyenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140228011301.GG26981@jama \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.