[bitbake-devel] [RFC PATCH 1/1] cooker: store a list of parsed configuration files
Joshua Lock
josh at linux.intel.com
Tue Aug 2 18:36:40 UTC 2011
On Tue, 2011-08-02 at 14:37 +0100, Richard Purdie wrote:
> On Mon, 2011-08-01 at 16:22 -0700, Joshua Lock wrote:
> > Signed-off-by: Joshua Lock <josh at 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?
I see them, that does seem like a more explicit and therefore cleaner
way of achieving this.
There'll be a series out shortly inspired by that and using an internal
namespace on the variable.
Cheers,
Joshua
--
Joshua Lock
Yocto Project "Johannes factotum"
Intel Open Source Technology Centre
More information about the bitbake-devel
mailing list