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


--
_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

Reply via email to