[OE-core] [CONSOLIDATED PULL 012/113] base.bbclass: Allow buildstats to be optionally supplied

Phil Blundell philb at gnu.org
Tue Jan 3 12:14:45 UTC 2012


On Mon, 2012-01-02 at 22:18 -0800, Saul Wold wrote:
> From: Mark Hatle <mark.hatle at windriver.com>
> 
> Buildstats should be allowed to be optionally enabled.  It's
> recommended that it be enabled via the USER_CLASSES setting.
> 
> Alternatively it could be enabled via the INHERIT_DISTRO or
> similar mechanism.
> 
> Signed-off-by: Mark Hatle <mark.hatle at windriver.com>

I don't think the short summary of this patch gives a very clear
indication of what it's doing.  The terminology "optionally supplied"
makes it sound as though you're talking about some sort of add-on data
file which can be enabled.  Whereas, what the patch seems actually to be
doing is removing the unconditional inherit of buildstats (i.e. turning
it off for almost everyone who has it on today) and adding a suggestion
in local.conf.sample as to how it might be turned on again.

(That said, I do think the intent of this patch is a good one; it's been
a long-standing source of irritation to me that disabling buildstats is
so awkward at present.)

p.






More information about the Openembedded-core mailing list