Re: [OE-core] [PATCH 0/7] Remove 'extend_recipe_sysroot' from 'BB_HASHEXCLUDE_COMMON'

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Sat, 2026-04-18 at 20:34 +0200, Adam Blank via lists.openembedded.org wrote:
> It has lain there for a while, obscuring certain aspects of
> how task signatures are calculated.
> 
> This changeset does not change the current status quo, but
> rather changes the way in which it is implemented. It can be
> followed by changes which further refine what is excluded and
> how, from those tasks which refer to 'extend_recipe_sysroot'.
> 
> Related discussion:
> https://lore.kernel.org/openembedded-core/aa893eb861bf4dd18e3e84e6bbebdee3b0367d1b.camel@linuxfoundation.org/
> 
> Signed-off-by: Adam Blank <adam.blank.g@gmail.com>
> ---
> Adam Blank (7):
>       package_pkgdata: fix typo to stop calling undefined function
>       staging: add 'vardepsexclude' to 'staging_populate_sysroot_dir'
>       staging: add 'extend_recipe_sysroot' to 'vardepsexclude'
>       cross: add 'extend_recipe_sysroot' to 'vardepsexclude'
>       native: add 'extend_recipe_sysroot' to 'vardepsexclude'
>       wic-tool: add 'extend_recipe_sysroot' to 'vardepsexclude'
>       bitbake.conf: remove 'extend_recipe_sysroot' from BB_HASHEXCLUDE_COMMON
> 
>  meta/classes-global/package_pkgdata.bbclass | 4 +++-
>  meta/classes-global/staging.bbclass         | 5 ++++-
>  meta/classes-recipe/cross.bbclass           | 1 +
>  meta/classes-recipe/native.bbclass          | 1 +
>  meta/conf/bitbake.conf                      | 2 +-
>  meta/recipes-core/meta/wic-tools.bb         | 1 +
>  6 files changed, 11 insertions(+), 3 deletions(-)

I finally found a bit of time to look at this. I took some of it, I
ended up reworking some of the patches as they needed rewording to
justify them and explain the changes being made.

I didn't take the extend_recipe_sysroot removal from bitbake.conf as
I'm still not convinced we have all the issues that might cause
identified. The function name typo and the incorrect vardepsexclude are
fixed though (or will be once a couple of patches pass testing/review
and merge).

Cheers,

Richard