[OE-core] [PATCH] classes/insane: add check for PN in OVERRIDES

Phil Blundell pb at pbcl.net
Thu May 9 16:37:33 UTC 2013


On Thu, 2013-05-09 at 17:26 +0100, Paul Eggleton wrote:
> If a recipe is named such that its PN value matches something already in
> OVERRIDES (such as when PN happens to be the same as MACHINE or DISTRO),
> it can have unexpected consequences. Assignments such as
> FILES_${PN} = "xyz" effectively turn into FILES = "xyz".

I think this patch is a fine one, but it makes me wonder whether we
should try to come up with a roadmap for changing the existing,
non-namespaced OVERRIDES into something less prone to namespace
collisions.  As a first step, we could just add namespaced equivalents
to the OVERRIDES in oe-core, i.e.:

MACHINEOVERRIDES ?= "${MACHINE}:machine-${MACHINE}"

etc, and then convert any usages of those within oe-core to use the new
style.  Then at some later date we can come up with a plan to remove the
old ones.

p.






More information about the Openembedded-core mailing list