From: Randy MacLeod <randy.macleod@windriver.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: DISTRO_FEATURES and world builds.
Date: Fri, 4 Oct 2013 16:56:01 -0400 [thread overview]
Message-ID: <524F2B61.10500@windriver.com> (raw)
In-Reply-To: <1380883572.18603.591.camel@ted>
On 13-10-04 06:46 AM, Richard Purdie wrote:
> On Thu, 2013-10-03 at 22:15 -0400, Randy MacLeod wrote:
>
>> If I drop x11 from the DF list things are even worse.
>>
>> I tried to sprinkle these recipes with:
>> REQUIRED_DISTRO_FEATURES = "opengl"
>> but just as I thought, that didn't work either.
>
> Did you also inherit the class that implements that?
Oops, thanks.
I got this to at least start compiling but now
I think it should be a oe-core-1.6 feature. I've opened:
https://bugzilla.yoctoproject.org/show_bug.cgi?id=5321
Details below.
// Randy
So that's a much cleaner change, eg:
meta/recipes-graphics/mesa/libglu_9.0.0.bb
...
-inherit autotools pkgconfig
+inherit autotools pkgconfig distro_features_check
+REQUIRED_DISTRO_FEATURES = "opengl"
and adding additional features is easy too.
I stuck with this syntax and fixed all the recipes when
they failed at initial parse time. That left a few packagegroups
to be fixed and I ended up with:
recipes-core/.../packagegroup-core-tools-testapps.bb | 2
recipes-core/.../packagegroup-self-hosted.bb | 14 ---
recipes-extended/.../packagegroup-core-lsb.bb | 41 ----------
recipes-graphics/.../packagegroup-core-clutter.bb | 3
The summary of this change was some simple changes:
meta/recipes-graphics/packagegroups/packagegroup-core-clutter.bb
...
-inherit packagegroup
+inherit packagegroup distro_features_check
+REQUIRED_DISTRO_FEATURES = "opengl"
and some refactoring of packagegroups to ensure that the
graphical part of lsb and self-hosted could be easily disabled.
--
# Randy MacLeod. SMTS, Linux, Wind River
Direct: 613.963.1350
prev parent reply other threads:[~2013-10-04 20:56 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-04 2:15 DISTRO_FEATURES and world builds Randy MacLeod
2013-10-04 4:28 ` Randy MacLeod
2013-10-04 8:02 ` Burton, Ross
2013-10-04 10:17 ` Martin Jansa
2013-10-04 10:46 ` Richard Purdie
2013-10-04 20:56 ` Randy MacLeod [this message]
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=524F2B61.10500@windriver.com \
--to=randy.macleod@windriver.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=richard.purdie@linuxfoundation.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