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

[Bruce Ashfield]

On 2018-12-12 8:36 p.m., Hongzhi, Song wrote:
> Thanks.
> 
> So would you help to remedy the commit log while merging the patch?

I've tweaked the message and added this to my queue. It will
come out early this week.

Bruce

> 
> 
> --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~
>>>
>>
>>
>