[PATCH] Adjust grep pattern for finding Kconfig symbols

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This script detects unknown Kconfig symbols by checking whether the
Kconfig macros used in the various *.config files have a corresponding
Kconfig symbol in the tree. Its core test is done with grep().

It turns out that match pattern used in grep() is too broad. For
instance, it doesn't report CONFIG_ACPI_PROCFS as unknown because there
is a Kconfig entry for ACPI_PROCFS_POWER in the tree. Make the pattern
match Kconfig symbols exactly to correct this.

Signed-off-by: Paul Bolle <pebolle@xxxxxxxxxx>
---
 scripts/check-configs.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/check-configs.pl b/scripts/check-configs.pl
index 39df8f4ad002..c74acf1d4c22 100644
--- a/scripts/check-configs.pl
+++ b/scripts/check-configs.pl
@@ -74,7 +74,7 @@ sub main {
 		my (@tmp) = parse_shipped( $shipped );
 		foreach my $ref ( @tmp ) {
 			say( STDERR "$shipped:$ref->[0]: No Kconfig symbol matches 'CONFIG_$ref->[1]'" )
-				unless (grep( /$ref->[1]/, keys( %configs )));
+				unless (grep( /^$ref->[1]$/, keys( %configs )));
 		}
 	}
 
-- 
2.17.2
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux