[bitbake-devel] [PATCH] data_smart: Ensure variable flags are accounted for in config data hash

[Martin Jansa]

Can we get this backported into bitbake-1.18 please?


On Thu, Jun 20, 2013 at 10:53 PM, Richard Purdie <
richard.purdie at linuxfoundation.org> wrote:

> Currently if the flags set against a variable in the base data store
> change, it doesn't automatically trigger a reparse when it really
> should. For example with the blacklist class setting:
>
> PNBLACKLIST[qemu] = "bar"
> PNBLACKLIST[bash] = "foo"
>
> will not trigger a reparse if only one entry is changed and a
> blacklisted recipe can still be built.
>
> I did consider using BB_SIGNATURE_EXCLUDE_FLAGS in here however it
> doesn't make sense, we want to trigger a reparse when any of the
> flags change too (which is different to the sstate signatures which
> we wouldn't want to change in those cases).
>
> [YOCTO #4627]
>
> Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
> ---
> diff --git a/bitbake/lib/bb/data_smart.py b/bitbake/lib/bb/data_smart.py
> index 2fd8ccd..fa7811e 100644
> --- a/bitbake/lib/bb/data_smart.py
> +++ b/bitbake/lib/bb/data_smart.py
> @@ -738,6 +738,12 @@ class DataSmart(MutableMapping):
>              value = d.getVar(key, False) or ""
>              data.update({key:value})
>
> +            varflags = d.getVarFlags(key)
> +            if not varflags:
> +                continue
> +            for f in varflags:
> +                data.update({'%s[%s]' % (key, f):varflags[f]})
> +
>          for key in ["__BBTASKS", "__BBANONFUNCS", "__BBHANDLERS"]:
>              bb_list = d.getVar(key, False) or []
>              bb_list.sort()
>
>
> _______________________________________________
> bitbake-devel mailing list
> bitbake-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/bitbake-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/bitbake-devel/attachments/20130708/0a0ace9f/attachment-0002.html>