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


Bruce Ashfield <bruce.ashfield@...>
 

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

Join yocto@lists.yoctoproject.org to automatically receive all group messages.