All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: "Cooper Jr., Franklin" <fcooper@ti.com>
Cc: "meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [PATCH 3/5] arago-prefs: make ARAGO_QT_PROVIDER aware	of Qt5
Date: Tue, 3 Jun 2014 11:55:33 -0400	[thread overview]
Message-ID: <20140603155533.GC21819@edge> (raw)
In-Reply-To: <8F29D6B095ED194EA1980491A5E029710C6C22A7@DFLE08.ent.ti.com>

On Tue, Jun 03, 2014 at 02:48:12PM +0000, Cooper Jr., Franklin wrote:
> Currently we also use ARAGO_QT_PROVIDER to determine if we need to include 
> GLES based applications or not specifically quick-playground.

Right now GLES side of Qt5 apps is not complete and quick-playground is not 
ported, so it doesn't make any difference. Once more and more apps are ported 
to Qt5, we can look into that.


> We also use it to set QT_SGX_SUPPORT in 
> packagegroup-arago-qte-toolchain-target.bb and packagegroup-arago-qte.bb so 
> maybe we need to pull this variable to arago-prefs.inc make it a boolean 
> variable. This way we can still conditional set it to TRUE depending on the 
> Qt 4 version we are using and always set it to true for Qt 5.

Those packagegroups are Qt4 specific and it's still Ok to use that variable 
for that purpose. Those packagegroups won't be included when building with Qt5


> Once we fixup the QT_SGX_SUPPORT variable then we can use that variable to 
> determine if a gles application should be pulled in or not.
> 
> 
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > Sent: Monday, June 02, 2014 9:30 PM
> > To: meta-arago@arago-project.org
> > Subject: [meta-arago] [PATCH 3/5] arago-prefs: make ARAGO_QT_PROVIDER
> > aware of Qt5
> > 
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > As ARAGO_QT_PROVIDER is used to select between gles and non-gles versions
> > of Qt4, make it aware of Qt5 and set it empty in that case.
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  meta-arago-distro/conf/distro/include/arago-prefs.inc | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-
> > arago-distro/conf/distro/include/arago-prefs.inc
> > index 6ae11e9..0885acf 100644
> > --- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
> > +++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> > @@ -25,7 +25,7 @@ PREFERRED_VERSION_libdrm = "2.4.46"
> > 
> >  PREFERRED_PROVIDER_virtual/gettext = "gettext"
> > 
> > -ARAGO_QT_PROVIDER = "${@base_contains('MACHINE_FEATURES','sgx','qt4-
> > embedded-gles','qt4-embedded',d)}"
> > +ARAGO_QT_PROVIDER =
> > "${@base_contains('QT_PROVIDER','qt5','',base_contains('MACHINE_FEATURES
> > ','sgx','qt4-embedded-gles','qt4-embedded',d),d)}"
> >  PREFERRED_PROVIDER_qt4-embedded = "${ARAGO_QT_PROVIDER}"
> > 
> >  PREFERRED_VERSION_qt4-embedded = "4.8.5"
> > --
> > 1.9.2
> > 
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


  reply	other threads:[~2014-06-03 15:55 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-03  2:29 [PATCH 0/5] qt5-related fixes Denys Dmytriyenko
2014-06-03  2:29 ` [PATCH 1/5] qt-tstat: fix install with Qt5, broken by latest change Denys Dmytriyenko
2014-06-03 14:33   ` Cooper Jr., Franklin
2014-06-03  2:29 ` [PATCH 2/5] refresh-screen: convert to Qt5 Denys Dmytriyenko
2014-06-03 14:39   ` Cooper Jr., Franklin
2014-06-03 14:47     ` Denys Dmytriyenko
2014-06-04 15:38       ` Denys Dmytriyenko
2014-06-04 16:29         ` Cooper Jr., Franklin
2014-06-03  2:29 ` [PATCH 3/5] arago-prefs: make ARAGO_QT_PROVIDER aware of Qt5 Denys Dmytriyenko
2014-06-03 14:48   ` Cooper Jr., Franklin
2014-06-03 15:55     ` Denys Dmytriyenko [this message]
2014-06-03  2:29 ` [PATCH 4/5] packagegroup-*-qt5: add Qt5 versions of arago packagegroups Denys Dmytriyenko
2014-06-03  2:29 ` [PATCH 5/5] tisdk-rootfs-image: use correct packagegroup for Qt4/Qt5 Denys Dmytriyenko
2014-06-03 14:40   ` Cooper Jr., Franklin

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=20140603155533.GC21819@edge \
    --to=denys@ti.com \
    --cc=fcooper@ti.com \
    --cc=meta-arago@arago-project.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.