Hi Joe,
2015-05-16 6:40 GMT+09:00 Joe Hershberger <joe.hershber...@ni.com>:
> Some config.h files live in arch and board directories. They will need
> to be cleaned up as well, so run the same filters there.
>
> Signed-off-by: Joe Hershberger <joe.hershber...@ni.com>
>
> ---
>
> Changes in v4:
> -New for version 4
>
> Changes in v3: None
> Changes in v2: None
>
> tools/moveconfig.py | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/tools/moveconfig.py b/tools/moveconfig.py
> index b6db058..25cee21 100755
> --- a/tools/moveconfig.py
> +++ b/tools/moveconfig.py
> @@ -352,6 +352,16 @@ def cleanup_headers(config_attrs, dry_run):
> if not fnmatch.fnmatch(filename, '*~'):
> cleanup_one_header(os.path.join(dirpath, filename), patterns,
> dry_run)
> + for (dirpath, dirnames, filenames) in os.walk('arch'):
> + for filename in filenames:
> + if not fnmatch.fnmatch(filename, '*~'):
> + cleanup_one_header(os.path.join(dirpath, filename), patterns,
> + dry_run)
> + for (dirpath, dirnames, filenames) in os.walk('board'):
> + for filename in filenames:
> + if not fnmatch.fnmatch(filename, '*~'):
> + cleanup_one_header(os.path.join(dirpath, filename), patterns,
> + dry_run)
To reduce code duplication, can we write like this or something?
for dir in 'include', 'arch', 'board':
for (dirpath, dirnames, filenames) in os.walk(dir):
for filename in filenames:
if not fnmatch.fnmatch(filename, '*~'):
cleanup_one_header(os.path.join(dirpath, filename),
patterns, dry_run)
--
Best Regards
Masahiro Yamada
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
