On Tue, 2026-03-17 at 12:09 +0000, Jose Quaresma via lists.openembedded.org
wrote:
> From: Jose Quaresma <[email protected]>
>
> The warnings about the hash equivalence are only valid when it the hash
> equivalence
> signature is being used, and for that we have to check the
> BB_SIGNATURE_HANDLER variable.
> We also have to consider the case where the variable BB_HASHSERVE is not
> defined.
>
> This fix the warning that is displayed when using the basic signature hash,
> OEBasicHash.
>
> > WARNING:
> > You are using a local hash equivalence server but have configured an sstate
> > mirror.
> > This will likely mean no sstate will match from the mirror.
> > You may wish to disable the hash equivalence use (BB_HASHSERVE),
> > or use a hash equivalence server alongside the sstate mirror.
>
> Signed-off-by: Jose Quaresma <[email protected]>
> ---
> meta/classes-global/sanity.bbclass | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/classes-global/sanity.bbclass
> b/meta/classes-global/sanity.bbclass
> index a170c3f5f5..e8eab1cdfe 100644
> --- a/meta/classes-global/sanity.bbclass
> +++ b/meta/classes-global/sanity.bbclass
> @@ -1007,8 +1007,8 @@ def check_sanity_everybuild(status, d):
> check_symlink(mirror_base, d)
>
> # Check sstate mirrors aren't being used with a local hash server and no
> remote
> - hashserv = d.getVar("BB_HASHSERVE")
> - if d.getVar("SSTATE_MIRRORS") and hashserv and
> hashserv.startswith("unix://") and not d.getVar("BB_HASHSERVE_UPSTREAM"):
> + hashserv = "" if d.getVar("BB_SIGNATURE_HANDLER") != "OEEquivHash" else
> d.getVar("BB_HASHSERVE") or ""
> + if d.getVar("SSTATE_MIRRORS") and hashserv.startswith("unix://") and not
> d.getVar("BB_HASHSERVE_UPSTREAM"):
> bb.warn("You are using a local hash equivalence server but have
> configured an sstate mirror. This will likely mean no sstate will match from
> the mirror. You may wish to disable the hash equivalence use (BB_HASHSERVE),
> or use a hash equivalence server alongside the sstate mirror.")
Good catch, thanks.
Since we're in python, we don't need to make this all one line and I
suspect it might be more readable if you put the two conditions on
separate lines and add a level of indentation.
"" if d.getVar("BB_SIGNATURE_HANDLER") != "OEEquivHash" else
d.getVar("BB_HASHSERVE") or ""
isn't easily parsable.
Cheers,
Richard
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#233322):
https://lists.openembedded.org/g/openembedded-core/message/233322
Mute This Topic: https://lists.openembedded.org/mt/118362789/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
