[U-Boot] [PATCH 5/6] moveconfig: Support building a simple config database

Simon Glass
2017-08-13T22:03:44Z

Hi Chris,

On 16 May 2017 at 00:58, Chris Packham <judge.packham@gmail.com> wrote:
>
>
> On 15/05/2017 11:47 PM, "Simon Glass" <sjg@chromium.org> wrote:
>
> Add a -b option which scans all the defconfigs and builds a database of
> all the CONFIG options used by each. This is useful for querying later.
>
> At present this only works with the separate -b option, which does not
> move any configs. It would be possible to adjust the script to build the
> database automatically when moving configs, but this might not be useful
> as the database does not change that often.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> tools/moveconfig.py | 81
> ++++++++++++++++++++++++++++++++++++++++++++++++-----
> 1 file changed, 74 insertions(+), 7 deletions(-)
>

> if configs:
> cleanup_headers(configs, options)
> @@ -1418,5 +1477,13 @@ def main():
> msg += '\n\nRsync all defconfig files using moveconfig.py'
> subprocess.call(['git', 'commit', '-s', '-m', msg])
>
> + if options.build_db:
> + with open(CONFIG_DATABASE, 'w') as fd:
> + for defconfig, configs in config_db.iteritems():
> + print >>fd, '%s' % defconfig
>
>
> Not sure how much we care but the python folks would say
>
> fd.write("%s\n" % defconfig)
>
> is more natural and will work for python 2 and 3.
>
> + for config in sorted(configs.keys()):
> + print >>fd, ' %s=%s' % (config, configs[config])
> + print >>fd
> +
> if __name__ == '__main__':
> main()
> --
> 2.13.0.rc2.291.g57267f2277-goog
>
>

Unfortunately I missed this, but I just sent a fixup patch.

Regards,
Simon