mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-04-20 19:45:12 +00:00
Meta: Always check completeness of ALL_THE_DEBUG_MACROS
This commit is contained in:
parent
e1baf9ec92
commit
e85aad6acc
Notes:
sideshowbarker
2024-07-19 01:07:52 +09:00
Author: https://github.com/BenWiederhake Commit: https://github.com/SerenityOS/serenity/commit/e85aad6acc0 Pull-request: https://github.com/SerenityOS/serenity/pull/4204
2 changed files with 37 additions and 0 deletions
36
Meta/check-debug-flags.sh
Executable file
36
Meta/check-debug-flags.sh
Executable file
|
@ -0,0 +1,36 @@
|
|||
#!/bin/bash
|
||||
|
||||
set -eo pipefail
|
||||
|
||||
script_path=$(cd -P -- "$(dirname -- "$0")" && pwd -P)
|
||||
cd "${script_path}/.."
|
||||
|
||||
MISSING_FLAGS=n
|
||||
|
||||
while IFS= read -r FLAG; do
|
||||
# We simply search whether the CMakeLists.txt *ever* sets the flag.
|
||||
# There are (basically) no false positives, but there might be false negatives,
|
||||
# for example we intentionally don't check for commented-out lines here.
|
||||
if ! grep -qP "set\(CMAKE_CXX_FLAGS \".* -D${FLAG}" CMakeLists.txt ; then
|
||||
echo "ALL_THE_DEBUG_MACROS probably doesn't include ${FLAG}"
|
||||
MISSING_FLAGS=y
|
||||
fi
|
||||
done < <(
|
||||
git ls-files -- \
|
||||
'*.cpp' \
|
||||
'*.h' \
|
||||
':!:Kernel/FileSystem/ext2_fs.h' \
|
||||
':!:Libraries/LibELF/exec_elf.h' \
|
||||
| xargs grep -P '^ *#.*DEBUG' \
|
||||
| sed -re 's,^.*[^a-zA-Z0-9_]([a-zA-Z0-9_]*DEBUG[a-zA-Z0-9_]*).*$,\1,' \
|
||||
| sort \
|
||||
| uniq
|
||||
)
|
||||
|
||||
if [ "n" != "${MISSING_FLAGS}" ] ; then
|
||||
echo "Some flags are missing for the ALL_THE_DEBUG_MACROS feature in CMakeLists.txt."
|
||||
echo "If you just added a new SOMETHING_DEBUG flag, that's great!"
|
||||
echo "We want to enable all of these in automated builds, so that the code doesn't rot."
|
||||
echo "Please add it to the ALL_THE_DEBUG_MACROS section."
|
||||
exit 1
|
||||
fi
|
|
@ -6,6 +6,7 @@ script_path=$(cd -P -- "$(dirname -- "$0")" && pwd -P)
|
|||
cd "${script_path}/.." || exit 1
|
||||
|
||||
for cmd in \
|
||||
Meta/check-debug-flags.sh \
|
||||
Meta/check-style.sh \
|
||||
Meta/lint-executable-resources.sh \
|
||||
Meta/lint-ipc-ids.sh \
|
||||
|
|
Loading…
Add table
Reference in a new issue