summaryrefslogtreecommitdiff
path: root/zephyr/zmake/zmake/zmake.py
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2021-03-28 15:55:40 +1300
committerCommit Bot <commit-bot@chromium.org>2021-03-31 00:57:05 +0000
commit3a86acf2b4ed5ca2932c442b67d178ea0f09b2f4 (patch)
treed6b1ddc83d309d942d28b12ab6846fb899f122b3 /zephyr/zmake/zmake/zmake.py
parent425882e0474be997976d6c9bef03213d91213c15 (diff)
downloadchrome-ec-3a86acf2b4ed5ca2932c442b67d178ea0f09b2f4.tar.gz
zephyr: zmake: Drop common cmake and ninja messages
When ninja gets an error it reports that the build stops, but we know that since we see the error and get a return code. Move it to DEBUG. Similarly cmake shows FAILED which is not useful since we can see the actual error immediately above, so move it to INFO. Also it prints out the command that failed but this is normally just a 20-line compiler invocation command which is not actionable and makes it easy to miss the one-line, useful error from the compiler. Move that to DEBUG. BUG=b:177096315 BRANCH=none TEST=manually test by running zmake build / configure Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I0a5946e0210c05f09c0af06817fd263ee17384f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2789794
Diffstat (limited to 'zephyr/zmake/zmake/zmake.py')
-rw-r--r--zephyr/zmake/zmake/zmake.py10
1 files changed, 10 insertions, 0 deletions
diff --git a/zephyr/zmake/zmake/zmake.py b/zephyr/zmake/zmake/zmake.py
index 99849f5796..066d853fde 100644
--- a/zephyr/zmake/zmake/zmake.py
+++ b/zephyr/zmake/zmake/zmake.py
@@ -53,6 +53,9 @@ def ninja_log_level_override(line, default_log_level):
# we don't care about entering directories since it happens every time
elif line.startswith("ninja: Entering directory"):
return logging.DEBUG
+ # we know the build stops from the compiler messages and ninja return code
+ elif line.startswith("ninja: build stopped"):
+ return logging.DEBUG
# someone prints a *** SUCCESS *** message which we don't need
elif line.startswith("***"):
return logging.DEBUG
@@ -62,6 +65,13 @@ def ninja_log_level_override(line, default_log_level):
# Try to drop output about the device tree
elif any(line.startswith(x) for x in cmake_suppress):
return logging.INFO
+ # this message is a bit like make failing. We already got the error output.
+ elif line.startswith("FAILED: CMakeFiles"):
+ return logging.INFO
+ # if a particular file fails it shows the build line used, but that is not
+ # useful except for debugging.
+ elif line.startswith("ccache"):
+ return logging.DEBUG
elif line.split()[0] in ["Memory", "FLASH:", "SRAM:", "IDT_LIST:"]:
pass
else: