aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Rini <trini@konsulko.com>2023-01-10 11:19:46 -0500
committerTom Rini <trini@konsulko.com>2023-01-20 12:27:24 -0500
commita03efb6f47b219612798c981cc3fddd653215244 (patch)
tree8161b6ce6ad6892591afe5be9b5db911981a6fee
parent6e7df1d151a7a127caf3b62ff6dfc003fc2aefcd (diff)
downloadu-boot-a03efb6f47b219612798c981cc3fddd653215244.zip
u-boot-a03efb6f47b219612798c981cc3fddd653215244.tar.gz
u-boot-a03efb6f47b219612798c981cc3fddd653215244.tar.bz2
CI: Make check for new defined CONFIG symbols even more robust
Now that all remaining in-tree cases where we define or undef a CONFIG symbol have been migrated to Kconfig or renamed to CFG we can make the CI check more robust. We will exclude the doc, tools and arch/arm/dts directories from this check as they are special cases. Further, we can exclude the scripts/kconfig/lkc.h and include/linux/kconfig.h files as the CONFIG values they define are special tooling cases and not real symbols. In the case of docs, the only places that currently fail this test are old documentation that should be rewritten so that we can remove this special case. Signed-off-by: Tom Rini <trini@konsulko.com> Reviewed-by: Simon Glass <sjg@chromium.org>
-rw-r--r--.azure-pipelines.yml3
-rw-r--r--.gitlab-ci.yml3
2 files changed, 4 insertions, 2 deletions
diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
index 49fc34f..5673bb7 100644
--- a/.azure-pipelines.yml
+++ b/.azure-pipelines.yml
@@ -64,7 +64,8 @@ stages:
# If grep succeeds and finds a match the test fails as we should
# have no matches.
- script: git grep -E '^#[[:blank:]]*(define|undef)[[:blank:]]*CONFIG_'
- include/configs `find arch -name config.h` && exit 1 || exit 0
+ :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
+ :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
- job: cppcheck
displayName: 'Static code analysis with cppcheck'
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 398fa2b..aaf9d25 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -131,7 +131,8 @@ check for new CONFIG symbols outside Kconfig:
# If grep succeeds and finds a match the test fails as we should
# have no matches.
- git grep -E '^#[[:blank:]]*(define|undef)[[:blank:]]*CONFIG_'
- include/configs `find arch -name config.h` && exit 1 || exit 0
+ :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
+ :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
# QA jobs for code analytics
# static code analysis with cppcheck (we can add --enable=all later)