Re: [RFC PATCH 1/1] cooker: store a list of parsed configuration files

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

On Mon, 2011-08-01 at 16:22 -0700, Joshua Lock wrote:
> Signed-off-by: Joshua Lock <josh@linux.intel.com>
> ---
>  lib/bb/cooker.py                     |    3 +++
>  lib/bb/parse/parse_py/ConfHandler.py |    3 +++
>  2 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/lib/bb/cooker.py b/lib/bb/cooker.py
> index 025dfe4..c7142e1 100644
> --- a/lib/bb/cooker.py
> +++ b/lib/bb/cooker.py
> @@ -772,6 +772,9 @@ class BBCooker:
>          for p in postfiles:
>              data = _parse(p, data)
>  
> +        self.configuration.parsed_configuration = (bb.data.getVar('PARSED_CONF', data) or '').split(':')
> +        bb.data.delVar('PARSED_CONF', data)
> +
>          # Handle any INHERITs and inherit the base class
>          bbclasses  = ["base"] + (data.getVar('INHERIT', True) or "").split()
>          for bbclass in bbclasses:
> diff --git a/lib/bb/parse/parse_py/ConfHandler.py b/lib/bb/parse/parse_py/ConfHandler.py
> index 102c0e9..22686b7 100644
> --- a/lib/bb/parse/parse_py/ConfHandler.py
> +++ b/lib/bb/parse/parse_py/ConfHandler.py
> @@ -103,6 +103,9 @@ def handle(fn, data, include):
>  
>      # DONE WITH PARSING... time to evaluate
>      bb.data.setVar('FILE', fn, data)
> +    conf_files = bb.data.getVar('PARSED_CONF', data) or ''
> +    bb.data.setVar('PARSED_CONF', "%s:%s" % (conf_files, abs_fn), data)
> +
>      statements.eval(data)
>      if oldfile:
>          bb.data.setVar('FILE', oldfile, data)

Hmm, this isn't particularly nice :/

At the very least call the variable __BB<something> to indicate its
internal.

Did you find and have a look at the internal __depends and
__base_depends variables?

Cheers,

Richard