[U-Boot] [PATCH v3 2/4] tools: moveconfig: cleanup whitelist entries

Chris Packham judge.packham at gmail.com
Tue May 2 09:30:47 UTC 2017


After moving to KConfig and removing from all headers options should be
removed from config_whitelist.txt so the build starts complaining if
someone adds them back.

Acked-by: Masahiro Yamada <yamada.masahiro at socionext.com>
Signed-off-by: Chris Packham <judge.packham at gmail.com>
---
Simon asked for it so here you go. I also thought about cleaning up the
README file references too but that will take a little bit of work to
parse reliably.

Changes in v3:
- Apply on top of cleanup patch

Changes in v2:
- Correct typo
- Collect ack from Masahiro

 tools/moveconfig.py | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/tools/moveconfig.py b/tools/moveconfig.py
index f8d4857..2361a43 100755
--- a/tools/moveconfig.py
+++ b/tools/moveconfig.py
@@ -600,6 +600,25 @@ def cleanup_extra_options(configs, options):
         cleanup_one_extra_option(os.path.join('configs', defconfig), configs,
                                  options)
 
+def cleanup_whitelist(configs, options):
+    """Delete config whitelist entries
+
+    Arguments:
+      configs: A list of CONFIGs to remove.
+      options: option flags.
+    """
+    if not confirm(options, 'Clean up whitelist entries?'):
+        return
+
+    with open(os.path.join('scripts', 'config_whitelist.txt')) as f:
+        lines = f.readlines()
+
+    lines = [x for x in lines if x.strip() not in configs]
+
+    with open(os.path.join('scripts', 'config_whitelist.txt'), 'w') as f:
+        f.write(''.join(lines))
+
+
 ### classes ###
 class Progress:
 
@@ -1296,6 +1315,7 @@ def main():
     if configs:
         cleanup_headers(configs, options)
         cleanup_extra_options(configs, options)
+        cleanup_whitelist(configs, options)
 
     if options.commit:
         subprocess.call(['git', 'add', '-u'])
-- 
2.10.1



More information about the U-Boot mailing list