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

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

On Fri, 2011-08-05 at 11:07 -0700, Joshua Lock wrote:
> On Fri, 2011-08-05 at 18:12 +0100, Richard Purdie wrote:
> > On Tue, 2011-08-02 at 11:51 -0700, Joshua Lock wrote:
> > >   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...

I'm not sure we need a variable just for .conf files. Anyone wanting
conf files can generate that information easily enough...

Cheers,

Richard