openbmc.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* meta-security and DISTRO_FEATURES
@ 2021-08-06 15:32 Bills, Jason M
  2021-08-06 15:40 ` Patrick Williams
  0 siblings, 1 reply; 3+ messages in thread
From: Bills, Jason M @ 2021-08-06 15:32 UTC (permalink / raw)
  To: openbmc

Hi All,

I am getting a warning about meta-security when building:
WARNING: You have included the meta-security layer, but 'security' has 
not been enabled in your DISTRO_FEATURES. Some bbappend files and 
preferred version setting may not take effect. See the meta-security 
README for details on enabling security support.

I checked in the README and it recommends adding 'DISTRO_FEATURES_append 
= " security"' to a config file to enable all the security features.

I did a quick grep on our layers and don't see that added anywhere. 
Does anyone know if this is something that we need or want to enable? 
If so, should we enable it in a common layer or will each individual 
layer decide whether to enable it?

Thanks!
-Jason

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: meta-security and DISTRO_FEATURES
  2021-08-06 15:32 meta-security and DISTRO_FEATURES Bills, Jason M
@ 2021-08-06 15:40 ` Patrick Williams
  2021-08-07  1:16   ` George Liu
  0 siblings, 1 reply; 3+ messages in thread
From: Patrick Williams @ 2021-08-06 15:40 UTC (permalink / raw)
  To: Bills, Jason M; +Cc: openbmc

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

On Fri, Aug 06, 2021 at 09:32:45AM -0600, Bills, Jason M wrote:
> Hi All,
> 
> I am getting a warning about meta-security when building:
> WARNING: You have included the meta-security layer, but 'security' has 
> not been enabled in your DISTRO_FEATURES. Some bbappend files and 
> preferred version setting may not take effect. See the meta-security 
> README for details on enabling security support.
> 
> I checked in the README and it recommends adding 'DISTRO_FEATURES_append 
> = " security"' to a config file to enable all the security features.
> 
> I did a quick grep on our layers and don't see that added anywhere. 
> Does anyone know if this is something that we need or want to enable? 
> If so, should we enable it in a common layer or will each individual 
> layer decide whether to enable it?

There is this pending commit to handle it at a meta-phosphor level.  I think
we're just waiting on a minor spelling fix in the variable name upstream.

https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/45322

-- 
Patrick Williams

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: meta-security and DISTRO_FEATURES
  2021-08-06 15:40 ` Patrick Williams
@ 2021-08-07  1:16   ` George Liu
  0 siblings, 0 replies; 3+ messages in thread
From: George Liu @ 2021-08-07  1:16 UTC (permalink / raw)
  To: Patrick Williams; +Cc: Bills, Jason M, openbmc

On Fri, Aug 6, 2021 at 11:41 PM Patrick Williams <patrick@stwcx.xyz> wrote:
>
> On Fri, Aug 06, 2021 at 09:32:45AM -0600, Bills, Jason M wrote:
> > Hi All,
> >
> > I am getting a warning about meta-security when building:
> > WARNING: You have included the meta-security layer, but 'security' has
> > not been enabled in your DISTRO_FEATURES. Some bbappend files and
> > preferred version setting may not take effect. See the meta-security
> > README for details on enabling security support.
> >
> > I checked in the README and it recommends adding 'DISTRO_FEATURES_append
> > = " security"' to a config file to enable all the security features.
> >
> > I did a quick grep on our layers and don't see that added anywhere.
> > Does anyone know if this is something that we need or want to enable?
> > If so, should we enable it in a common layer or will each individual
> > layer decide whether to enable it?
>
> There is this pending commit to handle it at a meta-phosphor level.  I think
> we're just waiting on a minor spelling fix in the variable name upstream.
>
> https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/45322

Yup, We have pushed a new patch to fix it.
https://lists.openembedded.org/g/openembedded-devel/message/92437

>
> --
> Patrick Williams

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-08-07  1:17 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-06 15:32 meta-security and DISTRO_FEATURES Bills, Jason M
2021-08-06 15:40 ` Patrick Williams
2021-08-07  1:16   ` George Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).