Skip to content

Commit 87f8f47

Browse files
committed
std.Build: Demote errors for exceeding max_rss to warnings.
We have no control over memory usage on arbitrary systems in the wild. But we would still like to get the warnings so we can adjust the values based on observations in the official ZSF CI. Closes #23254. Closes #23638.
1 parent e3b8aec commit 87f8f47

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

lib/compiler/build_runner.zig

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -599,7 +599,6 @@ fn prepare(
599599
if (run.max_rss_is_default) {
600600
std.debug.print("note: use --maxrss to override the default", .{});
601601
}
602-
return uncleanExit();
603602
}
604603
}
605604
}

lib/std/Build/Step.zig

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,6 @@ pub fn make(s: *Step, options: MakeOptions) error{ MakeFailed, MakeSkipped }!voi
247247
s.result_peak_rss, s.max_rss,
248248
}) catch @panic("OOM");
249249
s.result_error_msgs.append(arena, msg) catch @panic("OOM");
250-
return error.MakeFailed;
251250
}
252251
}
253252

0 commit comments

Comments
 (0)