From 87f8f47ba508b684d03c13b850cac3fb72efac26 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alex=20R=C3=B8nne=20Petersen?= Date: Mon, 2 Jun 2025 20:55:01 +0200 Subject: [PATCH] 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. --- lib/compiler/build_runner.zig | 1 - lib/std/Build/Step.zig | 1 - 2 files changed, 2 deletions(-) diff --git a/lib/compiler/build_runner.zig b/lib/compiler/build_runner.zig index 2cc3463e6600bce754a39f681befaa2d0f6d9268..1a019c80d96896441a5ca87384a6141c4bed3fcb 100644 --- a/lib/compiler/build_runner.zig +++ b/lib/compiler/build_runner.zig @@ -599,7 +599,6 @@ fn prepare( if (run.max_rss_is_default) { std.debug.print("note: use --maxrss to override the default", .{}); } - return uncleanExit(); } } } diff --git a/lib/std/Build/Step.zig b/lib/std/Build/Step.zig index 125b3e7947b698c876e020ffec51caa3180e4ac6..9cf0ca475ec394122e1cd9bcfd5733f63c1024aa 100644 --- a/lib/std/Build/Step.zig +++ b/lib/std/Build/Step.zig @@ -247,7 +247,6 @@ pub fn make(s: *Step, options: MakeOptions) error{ MakeFailed, MakeSkipped }!voi s.result_peak_rss, s.max_rss, }) catch @panic("OOM"); s.result_error_msgs.append(arena, msg) catch @panic("OOM"); - return error.MakeFailed; } } -- 2.54.0