[OE-core] [PATCH 1/1] layer_extra_sanity.bbclass: add new bbclass

[Otavio Salvador]

On Thu, Apr 16, 2015 at 7:31 AM, Martin Jansa <martin.jansa at gmail.com> wrote:
> On Thu, Apr 16, 2015 at 03:49:19PM +0800, Chen Qi wrote:
>> Add a new bbclass for extra sanity check for layers.
>>
>> These sanity checks include:
>> *) conf/machine and conf/distro don't appear in the same layer.
>>    The rational is that distro specific changes and BSP specific changes
>>    should be splitted into different layers for better maintenance.
>>    The meta layer is an exception.
>> *) A BSP layer is not included if MACHINE is not set to any of the conf file
>>    it provides.
>>    The rational is that BSP layer, by design, is supposed to make changes
>>    specific for the BSPs it covers. Including a BSP layer which doesn't contain
>>    a conf file matching the MACHINE setting might cause potential problem.
>
> This is exactly the opposite of what the manual says.
>
> BSP layer should be written in a way that it has no unwelcome
> side-effects for other MACHINES (by proper usage of MACHINE overrides).

Besides what Martin said it is very common to have extra BSP layers
for customer layers which are used for some machines. For example we
have customers which mix Freescale and Marvel machines in same
platform and all work very well.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750