Commit 4918c2ce2d739ba5e1b207a2ef32afcc04bfc435 | zig

author	Andrew Kelley <andrew@ziglang.org> 2024-05-31 12:39:08-07:00
committer	Andrew Kelley <andrew@ziglang.org> 2024-05-31 18:59:53-04:00
log	4918c2ce2d739ba5e1b207a2ef32afcc04bfc435
tree	efc061985ba14f1cef915b35f911dc49c62d7b4e
parent	c564a16a01811c18113fe304dbc3e8255af9f172

std.Build.Step.Run: global lock when child inherits stderr

The docs for setting stdio to "inherit" say:

It also means that this step will obtain a global lock to prevent other
steps from running in the meantime.

The implementation of this lock was missing but is now provided by this
commit.

closes #20119

1 files changed, 15 insertions(+), 9 deletions(-)

lib
- std
  - Build
    Step
    Run.zig

lib/std/Build/Step/Run.zig+15-9

 @@ -1241,20 +1241,26 @@ fn spawnChildAndCollect(
         child.stdin_behavior = .Pipe;
     }
-    if (run.stdio != .zig_test and !run.disable_zig_progress) {
+    const inherit = child.stdout_behavior == .Inherit or child.stderr_behavior == .Inherit;
+    if (run.stdio != .zig_test and !run.disable_zig_progress and !inherit) {
         child.progress_node = prog_node;
     }
-    try child.spawn();
+    const term, const result, const elapsed_ns = t: {
-    var timer = try std.time.Timer.start();
+        if (inherit) std.debug.lockStdErr();
+        defer if (inherit) std.debug.unlockStdErr();
-    const result = if (run.stdio == .zig_test)
+        try child.spawn();
-        evalZigTest(run, &child, prog_node)
+        var timer = try std.time.Timer.start();
-    else
-        evalGeneric(run, &child);
-    const term = try child.wait();
+        const result = if (run.stdio == .zig_test)
-    const elapsed_ns = timer.read();
+            evalZigTest(run, &child, prog_node)
+        else
+            evalGeneric(run, &child);
+        break :t .{ try child.wait(), result, timer.read() };
+    };
     return .{
         .stdio = try result,