Re: [meta-arm] [PATCH 1/5] layer.conf: add meta-python to LAYERDEPENDS for optee components

Re: [meta-arm] [PATCH 1/5] layer.conf: add meta-python to LAYERDEPENDS for optee components

[Denys Dmytriyenko]

On Wed, Apr 01, 2020 at 07:13:02PM -0400, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@ti.com>
> 
> optee components use pycrypto, pycryptodomex and pyelftools from meta-python layer

And this is quite unfortunate! As a proposed common dependency for ARM-based 
BSPs, I was hoping meta-arm could stay clean and only depend on OE-Core. While 
meta-python by itself may not be such a big deal, it is not a standalone layer 
and is part of a much larger meta-openembedded collection of layers. So, any 
BSP that wants to depend on meta-arm would now need to pull meta-openembedded 
in its entirety just to use meta-python...

-- 
Denys


> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  meta-arm/conf/layer.conf | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-arm/conf/layer.conf b/meta-arm/conf/layer.conf
> index 0b93e8d..d96e9f1 100644
> --- a/meta-arm/conf/layer.conf
> +++ b/meta-arm/conf/layer.conf
> @@ -9,5 +9,8 @@ BBFILE_COLLECTIONS += "meta-arm"
>  BBFILE_PATTERN_meta-arm = "^${LAYERDIR}/"
>  BBFILE_PRIORITY_meta-arm = "6"
>  
> -LAYERDEPENDS_meta-arm = "core"
> +LAYERDEPENDS_meta-arm = " \
> +    core \
> +    meta-python \
> +"
>  LAYERSERIES_COMPAT_meta-arm = "warrior zeus dunfell"
> -- 
> 2.7.4
> 

> 

Re: [meta-arm] [PATCH 1/5] layer.conf: add meta-python to LAYERDEPENDS for optee components

[Denys Dmytriyenko]

On Wed, Apr 01, 2020 at 07:20:26PM -0400, Denys Dmytriyenko wrote:
> On Wed, Apr 01, 2020 at 07:13:02PM -0400, Denys Dmytriyenko wrote:
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > optee components use pycrypto, pycryptodomex and pyelftools from meta-python layer
> 
> And this is quite unfortunate! As a proposed common dependency for ARM-based 
> BSPs, I was hoping meta-arm could stay clean and only depend on OE-Core. While 
> meta-python by itself may not be such a big deal, it is not a standalone layer 
> and is part of a much larger meta-openembedded collection of layers. So, any 
> BSP that wants to depend on meta-arm would now need to pull meta-openembedded 
> in its entirety just to use meta-python...

+Joshua


> -- 
> Denys
> 
> 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  meta-arm/conf/layer.conf | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/meta-arm/conf/layer.conf b/meta-arm/conf/layer.conf
> > index 0b93e8d..d96e9f1 100644
> > --- a/meta-arm/conf/layer.conf
> > +++ b/meta-arm/conf/layer.conf
> > @@ -9,5 +9,8 @@ BBFILE_COLLECTIONS += "meta-arm"
> >  BBFILE_PATTERN_meta-arm = "^${LAYERDIR}/"
> >  BBFILE_PRIORITY_meta-arm = "6"
> >  
> > -LAYERDEPENDS_meta-arm = "core"
> > +LAYERDEPENDS_meta-arm = " \
> > +    core \
> > +    meta-python \
> > +"
> >  LAYERSERIES_COMPAT_meta-arm = "warrior zeus dunfell"
> > -- 
> > 2.7.4
> > 
> 
> > 
> 

> 

Re: [meta-arm] [PATCH 1/5] layer.conf: add meta-python to LAYERDEPENDS for optee components

[Khem Raj]

On 5/11/20 10:28 AM, Denys Dmytriyenko wrote:
> On Wed, Apr 01, 2020 at 07:20:26PM -0400, Denys Dmytriyenko wrote:
>> On Wed, Apr 01, 2020 at 07:13:02PM -0400, Denys Dmytriyenko wrote:
>>> From: Denys Dmytriyenko <denys@ti.com>
>>>
>>> optee components use pycrypto, pycryptodomex and pyelftools from meta-python layer
>>

indeed unfortunate, some distros have actively avoided depending on 
meta-python and this derails that, can we somehow make these tools 
optional ? even if it is at a loss of features on OPTEE

>> And this is quite unfortunate! As a proposed common dependency for ARM-based
>> BSPs, I was hoping meta-arm could stay clean and only depend on OE-Core. While
>> meta-python by itself may not be such a big deal, it is not a standalone layer
>> and is part of a much larger meta-openembedded collection of layers. So, any
>> BSP that wants to depend on meta-arm would now need to pull meta-openembedded
>> in its entirety just to use meta-python...
> 
> +Joshua



> 
> 
>> -- 
>> Denys
>>
>>
>>> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>>> ---
>>>   meta-arm/conf/layer.conf | 5 ++++-
>>>   1 file changed, 4 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/meta-arm/conf/layer.conf b/meta-arm/conf/layer.conf
>>> index 0b93e8d..d96e9f1 100644
>>> --- a/meta-arm/conf/layer.conf
>>> +++ b/meta-arm/conf/layer.conf
>>> @@ -9,5 +9,8 @@ BBFILE_COLLECTIONS += "meta-arm"
>>>   BBFILE_PATTERN_meta-arm = "^${LAYERDIR}/"
>>>   BBFILE_PRIORITY_meta-arm = "6"
>>>   
>>> -LAYERDEPENDS_meta-arm = "core"
>>> +LAYERDEPENDS_meta-arm = " \
>>> +    core \
>>> +    meta-python \
>>> +"
>>>   LAYERSERIES_COMPAT_meta-arm = "warrior zeus dunfell"
>>> -- 
>>> 2.7.4
>>>
>>
>>>
>>
> 
>> 
> 

Re: [meta-arm] [PATCH 1/5] layer.conf: add meta-python to LAYERDEPENDS for optee components

[Peter Griffin]

[-- Attachment #1: Type: text/plain, Size: 921 bytes --]

On Thu, 2 Apr 2020 at 00:13, Denys Dmytriyenko <denis@denix.org> wrote:

> From: Denys Dmytriyenko <denys@ti.com>
>
> optee components use pycrypto, pycryptodomex and pyelftools from
> meta-python layer
>
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>

 Reviewed-by: Peter Griffin <peter.griffin@linaro.org>

---
>  meta-arm/conf/layer.conf | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/meta-arm/conf/layer.conf b/meta-arm/conf/layer.conf
> index 0b93e8d..d96e9f1 100644
> --- a/meta-arm/conf/layer.conf
> +++ b/meta-arm/conf/layer.conf
> @@ -9,5 +9,8 @@ BBFILE_COLLECTIONS += "meta-arm"
>  BBFILE_PATTERN_meta-arm = "^${LAYERDIR}/"
>  BBFILE_PRIORITY_meta-arm = "6"
>
> -LAYERDEPENDS_meta-arm = "core"
> +LAYERDEPENDS_meta-arm = " \
> +    core \
> +    meta-python \
> +"
>  LAYERSERIES_COMPAT_meta-arm = "warrior zeus dunfell"
> --
> 2.7.4
>
> 
>

[-- Attachment #2: Type: text/html, Size: 1671 bytes --]

Re: [meta-arm] [PATCH 1/5] layer.conf: add meta-python to LAYERDEPENDS for optee components

[Diego Sueiro]

> -----Original Message-----
> From: meta-arm@lists.yoctoproject.org <meta-arm@lists.yoctoproject.org>
> On Behalf Of Denys Dmytriyenko via lists.yoctoproject.org
> Sent: 02 April 2020 00:13
> To: meta-arm@lists.yoctoproject.org
> Cc: Denys Dmytriyenko <denys@ti.com>
> Subject: [meta-arm] [PATCH 1/5] layer.conf: add meta-python to
> LAYERDEPENDS for optee components
> 
> From: Denys Dmytriyenko <denys@ti.com>
> 
> optee components use pycrypto, pycryptodomex and pyelftools from meta-
> python layer
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>

Reviewed-by: Diego Sueiro <diego.sueiro@arm.com>

> ---
>  meta-arm/conf/layer.conf | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-arm/conf/layer.conf b/meta-arm/conf/layer.conf index
> 0b93e8d..d96e9f1 100644
> --- a/meta-arm/conf/layer.conf
> +++ b/meta-arm/conf/layer.conf
> @@ -9,5 +9,8 @@ BBFILE_COLLECTIONS += "meta-arm"
>  BBFILE_PATTERN_meta-arm = "^${LAYERDIR}/"
>  BBFILE_PRIORITY_meta-arm = "6"
> 
> -LAYERDEPENDS_meta-arm = "core"
> +LAYERDEPENDS_meta-arm = " \
> +    core \
> +    meta-python \
> +"
>  LAYERSERIES_COMPAT_meta-arm = "warrior zeus dunfell"
> --
> 2.7.4