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

[Simon Glass]

Hi Chris,

On 16 May 2017 at 00:58, Chris Packham <judge.packham at gmail.com> wrote:
>
>
> On 15/05/2017 11:47 PM, "Simon Glass" <sjg at 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 at 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