Re: [OE-core] [PATCH v2] insane.bbclass: Skip patches not in oe-core by full path

[Alexandre Belloni <alexandre.belloni@bootlin.com>]

Hello,

This doesn't seem to work as expected:

https://autobuilder.yoctoproject.org/typhoon/#/builders/120/builds/1599/steps/12/logs/stdio

https://autobuilder.yoctoproject.org/typhoon/#/builders/100/builds/3296/steps/12/logs/stdio

https://autobuilder.yoctoproject.org/typhoon/#builders/128/builds/495/steps/13/logs/stdio

On 15/08/2022 05:39:35+0000, Yang Xu via lists.openembedded.org wrote:
> The full path of patch may contain '/meta/' but not in oe-core, skip
> patches by checking it starts with oe-core full path or not.
> 
> Signed-off-by: Yang Xu <yang.xu@mediatek.com>
> ---
>  meta/classes-global/insane.bbclass | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass
> index 46ea41e271..42daf4388e 100644
> --- a/meta/classes-global/insane.bbclass
> +++ b/meta/classes-global/insane.bbclass
> @@ -1214,11 +1214,12 @@ python do_qa_patch() {
>      import re
>      from oe import patch
>  
> +    coremeta_path = os.path.join(d.getVar('COREBASE'), 'meta/')
>      for url in patch.src_patches(d):
>         (_, _, fullpath, _, _, _) = bb.fetch.decodeurl(url)
>  
>         # skip patches not in oe-core
> -       if '/meta/' not in fullpath:
> +       if not os.path.abspath(fullpath).startswith(os.path.abspath(coremeta_path)):
>             continue
>  
>         kinda_status_re = re.compile(r"^.*upstream.*status.*$", re.IGNORECASE | re.MULTILINE)
> -- 
> 2.25.1
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#169350): https://lists.openembedded.org/g/openembedded-core/message/169350
> Mute This Topic: https://lists.openembedded.org/mt/93030817/3617179
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com