From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Koen Kooi <koen@dominion.thruhere.net>
Cc: poky@yoctoproject.org,
Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [poky] [PATCH 3/3] meta-yocto: add pieces removed from oe-core for beagleboard & atom-pc
Date: Thu, 05 May 2011 13:21:15 +0100 [thread overview]
Message-ID: <1304598075.29269.2.camel@rex> (raw)
In-Reply-To: <40F9395F-41E1-4780-9E0C-6F0A91362CEB@dominion.thruhere.net>
On Thu, 2011-05-05 at 13:46 +0200, Koen Kooi wrote:
> Op 5 mei 2011, om 13:38 heeft Richard Purdie het volgende geschreven:
>
> > Hi Paul,
> >
> > Great work in doing this, thanks. I was just looking at it with a view
> > to making machine support cleaner and I think there are still things we
> > can likely to do help with this. As one example:
> >
> > On Wed, 2011-05-04 at 15:51 +0100, Paul Eggleton wrote:
> >> +++ b/meta-yocto/recipes-qt/qt4/qt4-x11-free_4.6.3.bbappend
> >> @@ -0,0 +1,2 @@
> >> +QT_GLFLAGS_atom-pc = "-opengl"
> >> +
> >> diff --git a/meta-yocto/recipes-qt/qt4/qt4-x11-free_4.7.2.bbappend b/meta-yocto/recipes-qt/qt4/qt4-x11-free_4.7.2.bbappend
> >> new file mode 100644
> >> index 0000000..076ade2
> >> --- /dev/null
> >> +++ b/meta-yocto/recipes-qt/qt4/qt4-x11-free_4.7.2.bbappend
> >> @@ -0,0 +1,2 @@
> >> +QT_GLFLAGS_atom-pc = "-opengl"
> >
> > could we set QT_GLFLAGS in the machine.conf file instead of using a
> > bbappend?
>
> Putting such USEFLAGS in machines sounds like a bad idea. In this case
> enabling it globally and falling back to mesa sw rendering at runtime
> is a better idea. The GL flag only enables extra API and libs, so it's
> good to have.
Agreed, longer term I think this is going to be the better way to handle
this. In this day and age, defaulting to sw rendering is probably the
sane thing to do.
Equally, moving this from a .bbappend to the machine file is a bit
cleaner too though :)
Cheers,
Richard
next prev parent reply other threads:[~2011-05-05 12:24 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-04 14:51 [PATCH 0/3] Remove machine/distro-specific metadata v3 + add some back to meta-yocto Paul Eggleton
2011-05-04 14:51 ` [PATCH 1/3] Remove machine-specific metadata for machines no longer in oe-core Paul Eggleton
2011-05-04 15:07 ` Gary Thomas
2011-05-04 15:21 ` [poky] " Paul Eggleton
2011-05-04 15:31 ` Gary Thomas
2011-05-05 11:46 ` Richard Purdie
2011-05-04 14:51 ` [PATCH 2/3] Remove distro-specific metadata for distros not " Paul Eggleton
2011-05-04 14:51 ` [PATCH 3/3] meta-yocto: add pieces removed from oe-core for beagleboard & atom-pc Paul Eggleton
2011-05-05 11:38 ` Richard Purdie
2011-05-05 11:46 ` Koen Kooi
2011-05-05 12:21 ` Richard Purdie [this message]
2011-05-05 12:31 ` [poky] " Koen Kooi
2011-05-05 12:59 ` Richard Purdie
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=1304598075.29269.2.camel@rex \
--to=richard.purdie@linuxfoundation.org \
--cc=koen@dominion.thruhere.net \
--cc=openembedded-core@lists.openembedded.org \
--cc=poky@yoctoproject.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox