[OE-core] [PATCH] meta/conf/layer.conf: Add recommended download layer

[Mark Hatle]

On 11/23/16 12:56 PM, Martin Jansa wrote:
> On Wed, Nov 23, 2016 at 12:38:42PM -0600, Mark Hatle wrote:
>> On 11/23/16 12:10 PM, Martin Jansa wrote:
>>> On Wed, Nov 23, 2016 at 11:42:09AM -0600, Mark Hatle wrote:
>>>> This is a Wind River specific patch and not generally applicable.
>>>
>>> Then why is it sent to oe-core ML?
>>
>> As noted in the cover letter, I'm required to by Yocto Project compliance
>> requirements.
>>
>> As indicated LAST time I got scolded, I was told to indicate this in the patch
>> summary email -- which I did.
> 
> Sorry I've noticed the cover letter only after the response.
> 
> So it's only because of this requirement from Yocto Project compliance?

yes.

> "Have all patches applied to BitBake and OpenEmbedded-Core (if present)
> been submitted to the open source community?"
> 
> Shouldn't the wording be change to something like "all applicable
> patches" or "all generally useful patches"?
> 
> It seems strange to send project specific patches together with cover
> saying that they aren't generally applicable and shouldn't be merged,
> just because of this requirement.

It was done this way to prevent people from cheating and claiming something was
(effectively) not useful/applicable/etc when it some special secret sauce.

Even these patches while not generally applicable could give someone else the
idea to duplicate (or improve) on our approach at providing layer specific
download layers.

--Mark

> Regards,
> 
>>>> Signed-off-by: Mark Hatle <mark.hatle at windriver.com>
>>>> ---
>>>>  meta/conf/layer.conf | 2 ++
>>>>  1 file changed, 2 insertions(+)
>>>>
>>>> diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
>>>> index 24b4df0..a94e524 100644
>>>> --- a/meta/conf/layer.conf
>>>> +++ b/meta/conf/layer.conf
>>>> @@ -11,6 +11,8 @@ BBFILE_PRIORITY_core = "5"
>>>>  # cause compatibility issues with other layers
>>>>  LAYERVERSION_core = "9"
>>>>  
>>>> +LAYERRECOMMENDS_core = "oe-core-dl-2-2"
>>>> +
>>>>  BBLAYERS_LAYERINDEX_NAME_core = "openembedded-core"
>>>>  
>>>>  # Set a variable to get to the top of the metadata location
>>>> -- 
>>>> 2.9.3
>>>>
>>>> -- 
>>>> _______________________________________________
>>>> Openembedded-core mailing list
>>>> Openembedded-core at lists.openembedded.org
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>>
>>
>