diff options
author | Simon Glass <sjg@chromium.org> | 2021-03-28 15:55:40 +1300 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-03-31 00:57:05 +0000 |
commit | 3a86acf2b4ed5ca2932c442b67d178ea0f09b2f4 (patch) | |
tree | d6b1ddc83d309d942d28b12ab6846fb899f122b3 /zephyr/zmake/zmake/zmake.py | |
parent | 425882e0474be997976d6c9bef03213d91213c15 (diff) | |
download | chrome-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.py | 10 |
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: |