Re: [PATCH 0/2] Store a list of parsed configuration files

[Joshua Lock <josh@linux.intel.com>]

On Fri, 2011-08-05 at 18:12 +0100, Richard Purdie wrote:
> On Tue, 2011-08-02 at 11:51 -0700, Joshua Lock wrote:
> > This series addresses Yocto #1246 "Configuration changes could be saved to a
> > file the cooker isn't using" by having the parser store a list of
> > configuration files it has parsed in the data object and then switching the
> > cooker to verify against this list that the found file was parsed before
> > emitting the ConfigFilePathFound event.
> > 
> > The following changes since commit 1009ca570a750a00b0e60afcc30ead070c7b310a:
> > 
> >   hob: remove temporary directory on program shutdown (2011-07-30 12:21:18 -0700)
> > 
> > are available in the git repository at:
> >   git://github.com/incandescant/bitbake configfiles
> >   https://github.com/incandescant/bitbake/tree/configfiles
> > 
> > Joshua Lock (2):
> >   bb/parse: track parsed configuration files in __BB_PARSED_CONF
> >   bb/cooker: only emit ConfigFilePathFound for files which were parsed
> 
> Am I right in thinking this series is now superseded by the tweak to
> include the information in __base_depends?

That's right. The only reason I haven't sent that patch out yet is
because I'm in two minds about which is the cleanest way.

Because __base_depends a) contains more than just .conf files and b)
gets renamed I'm thinking the cleanest thing to do would be to iterate
__depends before the call to renameVar, pull out all conf files and
store them in self.configuration.config_files (or similar).

I wasn't sure adding an extra list to the configuration object would be
appreciated though...

Cheers,
Joshua
-- 
Joshua Lock
        Yocto Project "Johannes factotum"
        Intel Open Source Technology Centre