fix: catch error infos when clang-tidy fails, also return 1 when fails.

Signed-off-by: WU MENGHUNG <[email protected]>
ament · Sep 8, 2023 · 15d5c2c · 15d5c2c
1 parent 6d1766f
commit 15d5c2c
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/ament_clang_tidy/ament_clang_tidy/main.py b/ament_clang_tidy/ament_clang_tidy/main.py
@@ -180,6 +180,7 @@ def start_subprocess(full_cmd):
                 print('The invocation of "%s" failed with error code %d: %s' %
                       (os.path.basename(clang_tidy_bin), e.returncode, e),
                       file=sys.stderr)
+                output = e.output.decode()
             return output
 
         files = []
@@ -268,6 +269,9 @@ def start_subprocess(full_cmd):
         with open(args.xunit_file, 'w') as f:
             f.write(xml)
 
+    if output:
+        sys.exit(1)
+
 
 def find_executable(file_names):
     paths = os.getenv('PATH').split(os.path.pathsep)