pw_presubmit: Drop extra lines from bazel summary

Drop unnecessary lines from the bazel failure summary. Bug: b/262262623 Change-Id: I9e345389bf0eb12bde75dcb03a6b735df89c3678 Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/125651 Reviewed-by: Alexei Frolov <[email protected]> Pigweed-Auto-Submit: Rob Mohr <[email protected]> Commit-Queue: Auto-Submit <[email protected]>
google · Dec 29, 2022 · 2c65b6c · 2c65b6c
1 parent 377722d
commit 2c65b6c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/pw_presubmit/py/bazel_parser_test.py b/pw_presubmit/py/bazel_parser_test.py
@@ -64,10 +64,8 @@
 
 _REAL_TEST_SUMMARY = """
 ERROR: /usr/local/google/home/mohrr/pigweed/pigweed/pw_kvs/BUILD.bazel:25:14: Compiling pw_kvs/entry.cc failed: (Exit 1): gcc failed: error executing command
-# Configuration: 752863e407a197a5b9da05cfc572e7013efd6958e856cee61d2fa474ed...
 # Execution platform: @local_config_platform//:host
 
-Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
 pw_kvs/entry.cc:49:20: error: no member named 'Dat' in 'pw::Status'
     return Status::Dat aLoss();
            ~~~~~~~~^

diff --git a/pw_presubmit/py/pw_presubmit/bazel_parser.py b/pw_presubmit/py/pw_presubmit/bazel_parser.py
@@ -42,6 +42,12 @@ def parse_bazel_stdout(bazel_stdout: Path) -> str:
                 if line.strip().startswith(('(cd /', 'exec env')):
                     continue
 
+                if line.strip().startswith('Use --sandbox_debug to see'):
+                    continue
+
+                if line.strip().startswith('# Configuration'):
+                    continue
+
                 # An "<ALLCAPS>:" line usually means compiler output is done
                 # and useful compiler errors are complete.
                 if re.match(r'^(?!ERROR)[A-Z]+:', line):