[coreboot-gerrit] New patch to review for coreboot: kconfig_lint: Skip temp files when looking for unused Kconfig files.

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Tue Jan 26 04:17:55 CET 2016


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13462

-gerrit

commit 9a9679515997e915a23f98caf903cf956a2112af
Author: Martin Roth <martinroth at google.com>
Date:   Mon Jan 25 16:45:14 2016 -0700

    kconfig_lint: Skip temp files when looking for unused Kconfig files.
    
    Don't warn on Kconfig.orig and Kconfig~ files when trying to verify
    that all the Kconfig files in the coreboot source tree are being loaded.
    
    Change-Id: Ie7babe60b29735e5ccc5f93f4e42ad82dfb47044
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 util/lint/kconfig_lint | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/util/lint/kconfig_lint b/util/lint/kconfig_lint
index 2fbaf97..bec86e4 100755
--- a/util/lint/kconfig_lint
+++ b/util/lint/kconfig_lint
@@ -1168,7 +1168,11 @@ sub print_wholeconfig {
 #-------------------------------------------------------------------------------
 sub check_if_file_referenced {
     my $filename = $File::Find::name;
-    if ( ( $filename =~ /Kconfig/ ) && ( !exists $loaded_files{$filename} ) ) {
+    if (   ( $filename =~ /Kconfig/ )
+        && ( !$filename =~ /\.orig$/ )
+        && ( !$filename =~ /~$/ )
+        && ( !exists $loaded_files{$filename} ) )
+    {
         show_warning("'$filename' is never referenced");
     }
 }



More information about the coreboot-gerrit mailing list