[yocto] [yocto-kernel-tools][PATCH] tools/kconf_check: modify grep pattern

[Hongzhi, Song]

Thanks.

So would you help to remedy the commit log while merging the patch?


--Hongzhi


On 12/13/2018 02:45 AM, Bruce Ashfield wrote:
> On 12/11/18 5:12 AM, Hongzhi.Song wrote:
>> The cmd line, <grep '^[ ]*\(menu\)*config '>, can't
>> match all expect config options.
>>
>> This is because that it is not always space after 'config'
>
> This should say "not always one space". There really should
> always just be a single space, but typos do sneak in. I'll
> queue the patch shortly.
>
> Bruce
>
>> in kernel-source/*/.../Kconfig.
>> e.g. "config  IP_VS_IPV6" in net/netfilter/ipvs/Kconfig
>>
>> So we should change the cmd to grep '^[  ]*\(menu\)*config\s'.
>>
>> Signed-off-by: Hongzhi.Song <hongzhi.song at windriver.com>
>> ---
>>   tools/kconf_check | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tools/kconf_check b/tools/kconf_check
>> index aa86180..20b49cd 100755
>> --- a/tools/kconf_check
>> +++ b/tools/kconf_check
>> @@ -241,7 +241,7 @@ find ${kernel_source} \
>>   # known list of all Kconfig* files.  Again, must filter dups.
>>   rm -f ${LOGDIR}/all.cfg
>>   for i in `cat ${LOGDIR}/all.kcf` ; do
>> -    cat ${kernel_source}/$i | grep '^[     ]*\(menu\)*config ' | \
>> +    cat ${kernel_source}/$i | grep '^[     ]*\(menu\)*config\s' | \
>>       awk '{print "CONFIG_"$2}' >> ${LOGDIR}/all.cfg
>>   done
>>   mv -f ${LOGDIR}/all.cfg ${LOGDIR}/all.cfg~
>>
>
>