commit 271fd8ffc3883b19ad51508773521ce6f726960d
author Simon Glass <sjg@chromium.org> Wed Nov 03 21:09:15 2021 -0600
committer Simon Glass <sjg@chromium.org> Sat Nov 13 08:16:39 2021 -0700
tree 3390f4ecfde18d923ff502adf1d4d9ef0c58fc30
parent 6db1e32534bddaa49f7c2b873a0213f1b78c3d75

binman: Report an error if test files fail to compile

At present any error from the 'make' command is silently swallowed by the
test system. Fix this by showing it when detected.

Signed-off-by: Simon Glass <sjg@chromium.org>

tools/binman/elf_test.py

diff --git a/tools/binman/elf_test.py b/tools/binman/elf_test.py
index 7a12801..bcccd78 100644
--- a/tools/binman/elf_test.py
+++ b/tools/binman/elf_test.py
@@ -70,8 +70,12 @@
     # correctly. So drop any make flags here.
     if 'MAKEFLAGS' in os.environ:
         del os.environ['MAKEFLAGS']
-    tools.Run('make', '-C', target_dir, '-f',
-              os.path.join(testdir, 'Makefile'), 'SRC=%s/' % testdir)
+    try:
+        tools.Run('make', '-C', target_dir, '-f',
+                  os.path.join(testdir, 'Makefile'), 'SRC=%s/' % testdir)
+    except ValueError as e:
+        # The test system seems to suppress this in a strange way
+        print(e)
 
 
 class TestElf(unittest.TestCase):