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

Bruce Ashfield bruce.ashfield at windriver.com
Wed Dec 12 10:45:02 PST 2018


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



More information about the yocto mailing list