Add nominal logging for test_all and style_all

in separate directories so they can easily be differentiated.
lasp · Sep 14, 2024 · 1ddb11c · 1ddb11c
1 parent c8f71f3
commit 1ddb11c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/redo/rules/build_style_all.py b/redo/rules/build_style_all.py
@@ -81,7 +81,9 @@ def build(self, redo_1, redo_2, redo_3):
 
         # Make a build directory at the top level:
         failed_style_log_dir = os.path.join(directory, "build" + os.sep + "failed_style_logs")
+        log_dir = os.path.join(directory, "build" + os.sep + "style_logs")
         filesystem.safe_makedir(failed_style_log_dir)
+        filesystem.safe_makedir(log_dir)
 
         # Run tests:
         exit_code = 0
@@ -96,7 +98,8 @@ def build(self, redo_1, redo_2, redo_3):
             sys.stderr.flush()
             database.setup.reset()
             try:
-                redo.redo([os.path.join(test, "style"), "2>/dev/null", "1>/dev/null"])
+                style_run_log = os.path.join(log_dir, rel_test.replace(os.sep, "_") + ".log")
+                redo.redo([os.path.join(test, "style"), "1>&2", "2>" + style_run_log])
 
                 # See if there is anything in the log file:
                 style_log = os.path.join(

diff --git a/redo/rules/build_test_all.py b/redo/rules/build_test_all.py
@@ -71,7 +71,7 @@ def build(self, redo_1, redo_2, redo_3):
 
         # Make a build directory at the top level:
         failed_test_log_dir = os.path.join(directory, "build" + os.sep + "failed_test_logs")
-        log_dir = os.path.join(directory, "build" + os.sep + "log")
+        log_dir = os.path.join(directory, "build" + os.sep + "test_logs")
         filesystem.safe_makedir(failed_test_log_dir)
         filesystem.safe_makedir(log_dir)