[bitbake-devel] [PATCH 0/2] Store a list of parsed configuration files
Joshua Lock
josh at linux.intel.com
Fri Aug 5 18:07:51 UTC 2011
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
More information about the bitbake-devel
mailing list