From: Arnout Vandecappelle (Essensium/Mind) Date: Thu, 1 Aug 2019 08:02:35 +0000 (+0200) Subject: utils/checkpackagelib: CommentsMenusPackagesOrder: fix 'menuconfig' handling X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c62a2829209671f4f7a209a07eca6608ebd0dfa7;p=buildroot.git utils/checkpackagelib: CommentsMenusPackagesOrder: fix 'menuconfig' handling The CommentsMenusPackagesOrder check builds the 'state' to track the depth of menus and conditions. However, a menuconfig doesn't create a menu by itself - it is always followed by a condition that implies the menu. As a result, when unwinding the 'state', the level will be wrong. Fix this by checking for menu followed by a space, so it no longer matches menuconfig. For consistency, do the same for comment and if as well. Signed-off-by: Arnout Vandecappelle (Essensium/Mind) Signed-off-by: Jerzy Grzegorek Signed-off-by: Thomas Petazzoni --- diff --git a/utils/checkpackagelib/lib_config.py b/utils/checkpackagelib/lib_config.py index 7d7567ac67..b42760396e 100644 --- a/utils/checkpackagelib/lib_config.py +++ b/utils/checkpackagelib/lib_config.py @@ -77,8 +77,8 @@ class CommentsMenusPackagesOrder(_CheckFunction): "package/Config.in.host"]: return - if text.startswith("comment") or text.startswith("if") or \ - text.startswith("menu"): + if text.startswith("comment ") or text.startswith("if ") or \ + text.startswith("menu "): if text.startswith("comment"): if not self.state.endswith("-comment"):