Merge pull request PowerDNS#17371 from rgacogne/error-when-clang-tidy-fails

ci: Error when clang tidy fails

Author: rgacogne

.github/scripts/clang-tidy-diff.py

@@ -51,7 +51,7 @@
     import queue as queue
 
 
-def run_tidy(task_queue, lock, timeout):
+def run_tidy(task_queue, lock, timeout, failed_files):
     watchdog = None
     while True:
         command = task_queue.get()
@@ -63,6 +63,14 @@ def run_tidy(task_queue, lock, timeout):
                 watchdog.start()
 
             stdout, stderr = proc.communicate()
+            if proc.returncode != 0:
+                if proc.returncode < 0:
+                    msg = "Terminated by signal %d : %s\n" % (
+                        -proc.returncode,
+                        " ".join(command),
+                    )
+                    stderr += msg.encode("utf-8")
+                failed_files.append(command)
 
             with lock:
                 sys.stdout.write(stdout.decode("utf-8") + "\n")
@@ -82,9 +90,9 @@ def run_tidy(task_queue, lock, timeout):
             task_queue.task_done()
 
 
-def start_workers(max_tasks, tidy_caller, task_queue, lock, timeout):
+def start_workers(max_tasks, tidy_caller, arguments):
     for _ in range(max_tasks):
-        t = threading.Thread(target=tidy_caller, args=(task_queue, lock, timeout))
+        t = threading.Thread(target=tidy_caller, args=arguments)
         t.daemon = True
         t.start()
 
@@ -219,8 +227,10 @@ def main():
     # A lock for console output.
     lock = threading.Lock()
 
+    # List of files with a non-zero return code.
+    failed_files = []
     # Run a pool of clang-tidy workers.
-    start_workers(max_task_count, run_tidy, task_queue, lock, args.timeout)
+    start_workers(max_task_count, run_tidy, (task_queue, lock, args.timeout, failed_files))
 
     # Form the common args list.
     common_clang_tidy_args = []
@@ -263,8 +273,12 @@ def main():
 
         task_queue.put(command)
 
+    # Application return code
+    return_code = 0
     # Wait for all threads to be done.
     task_queue.join()
+    if failed_files:
+        return_code = 1
 
     if yaml and args.export_fixes:
         print("Writing fixes to " + args.export_fixes + " ...")
@@ -273,9 +287,11 @@ def main():
         except Exception:
             sys.stderr.write("Error exporting fixes.\n")
             traceback.print_exc()
+            return_code = 1
 
     if tmpdir:
         shutil.rmtree(tmpdir)
+    sys.exit(return_code)
 
 
 if __name__ == "__main__":

.github/scripts/helpers.py

@@ -2,6 +2,7 @@
 
 import json
 import os
+import sys
 
 import git
 import yaml
@@ -43,6 +44,6 @@ def index_compdb(file_contents):
     """Index the compilation database."""
     result = set()
     for item in file_contents:
-        filename = os.path.join(item["directory"], item["file"])
+        filename = os.path.normpath(os.path.join(item["directory"], item["file"]))
         result.add(filename)
     return result