On Thu, Feb 15, 2024 at 12:15 PM Joe Slater via lists.yoctoproject.org
<joe.slater=windriver....@lists.yoctoproject.org> wrote:
>
> From: Joe Slater <joe.sla...@windriver.com>
>
> Introduce META_SECURITY_SANITY_CHECK_WARNING variable which
> can be overridden, if desired.

The existence of the patch implies that there's a reason why the warning message
isn't appropriate for your use case.

That's something that should be explained in the patch.

A knob to disable the warning if you know what you are doing has already been
provided. So again, this patch implies that you want the warning, but want to
change the message .. it would be good to understand why.

Bruce

>
> Signed-off-by: Joe Slater <joe.sla...@windriver.com>
> ---
>  classes/sanity-meta-security.bbclass | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/classes/sanity-meta-security.bbclass 
> b/classes/sanity-meta-security.bbclass
> index f9e2698..95180aa 100644
> --- a/classes/sanity-meta-security.bbclass
> +++ b/classes/sanity-meta-security.bbclass
> @@ -1,10 +1,15 @@
>  addhandler security_bbappend_distrocheck
>  security_bbappend_distrocheck[eventmask] = "bb.event.SanityCheck"
> +
>  python security_bbappend_distrocheck() {
>      skip_check = e.data.getVar('SKIP_META_SECURITY_SANITY_CHECK') == "1"
>      if 'security' not in e.data.getVar('DISTRO_FEATURES').split() and not 
> skip_check:
> -        bb.warn("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.")
> +        bb.warn(e.data.getVar('META_SECURITY_SANITY_CHECK_WARNING'))
>  }
> +
> +META_SECURITY_SANITY_CHECK_WARNING ??= "\
> +You have included the meta-security layer, but 'security' has not been \
> +enabled in your DISTRO_FEATURES. Some bbappend files and preferred version \
> +settings may not take effect. See the meta-security README for details on \
> +enabling security support."
> +
> --
> 2.25.1
>
>
> 
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#62500): https://lists.yoctoproject.org/g/yocto/message/62500
Mute This Topic: https://lists.yoctoproject.org/mt/104377037/21656
Group Owner: yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to