From e7dcf0b6ef10944271fea4b677cb2722c632dabd Mon Sep 17 00:00:00 2001 From: bsubei <6508762+bsubei@users.noreply.github.com> Date: Sun, 21 Jul 2024 12:23:30 -0400 Subject: [PATCH 1/2] fix build files broken by latest 0.14.0-dev changes to Build.Step.MakeOptions --- build.zig | 10 +++++----- test/tests.zig | 8 ++++---- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/build.zig b/build.zig index b40f8bd..8c7ab7b 100644 --- a/build.zig +++ b/build.zig @@ -274,7 +274,7 @@ const ZiglingStep = struct { return self; } - fn make(step: *Step, prog_node: std.Progress.Node) !void { + fn make(step: *Step, options: Step.MakeOptions) !void { // NOTE: Using exit code 2 will prevent the Zig compiler to print the message: // "error: the following build command failed with exit code 1:..." const self: *ZiglingStep = @alignCast(@fieldParentPtr("step", step)); @@ -285,7 +285,7 @@ const ZiglingStep = struct { return; } - const exe_path = self.compile(prog_node) catch { + const exe_path = self.compile(options.progress_node) catch { self.printErrors(); if (self.exercise.hint) |hint| @@ -295,7 +295,7 @@ const ZiglingStep = struct { std.process.exit(2); }; - self.run(exe_path.?, prog_node) catch { + self.run(exe_path.?, options.progress_node) catch { self.printErrors(); if (self.exercise.hint) |hint| @@ -436,7 +436,7 @@ const ZiglingStep = struct { zig_args.append("--listen=-") catch @panic("OOM"); - return try self.step.evalZigProcess(zig_args.items, prog_node); + return try self.step.evalZigProcess(zig_args.items, prog_node, false); } fn help(self: *ZiglingStep) void { @@ -525,7 +525,7 @@ const PrintStep = struct { return self; } - fn make(step: *Step, _: std.Progress.Node) !void { + fn make(step: *Step, _: Step.MakeOptions) !void { const self: *PrintStep = @alignCast(@fieldParentPtr("step", step)); print("{s}", .{self.message}); } diff --git a/test/tests.zig b/test/tests.zig index 126a1cd..9fed107 100644 --- a/test/tests.zig +++ b/test/tests.zig @@ -150,7 +150,7 @@ const CheckNamedStep = struct { return self; } - fn make(step: *Step, _: std.Progress.Node) !void { + fn make(step: *Step, _: Step.MakeOptions) !void { const b = step.owner; const self: *CheckNamedStep = @alignCast(@fieldParentPtr("step", step)); const ex = self.exercise; @@ -202,7 +202,7 @@ const CheckStep = struct { return self; } - fn make(step: *Step, _: std.Progress.Node) !void { + fn make(step: *Step, _: Step.MakeOptions) !void { const b = step.owner; const self: *CheckStep = @alignCast(@fieldParentPtr("step", step)); const exercises = self.exercises; @@ -325,7 +325,7 @@ const FailStep = struct { return self; } - fn make(step: *Step, _: std.Progress.Node) !void { + fn make(step: *Step, _: Step.MakeOptions) !void { const b = step.owner; const self: *FailStep = @alignCast(@fieldParentPtr("step", step)); @@ -368,7 +368,7 @@ const HealStep = struct { return self; } - fn make(step: *Step, _: std.Progress.Node) !void { + fn make(step: *Step, _: Step.MakeOptions) !void { const b = step.owner; const self: *HealStep = @alignCast(@fieldParentPtr("step", step)); From 4cc1158fbcc5380cf7d7b38cd63f04733701e7fa Mon Sep 17 00:00:00 2001 From: bsubei <6508762+bsubei@users.noreply.github.com> Date: Sun, 21 Jul 2024 12:26:03 -0400 Subject: [PATCH 2/2] fix tests build file broken by addRemoveDirTree now requiring LazyPath --- test/tests.zig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/tests.zig b/test/tests.zig index 9fed107..5f64f8e 100644 --- a/test/tests.zig +++ b/test/tests.zig @@ -50,7 +50,7 @@ pub fn addCliTests(b: *std.Build, exercises: []const Exercise) *Step { case_step.dependOn(&verify.step); } - const cleanup = b.addRemoveDirTree(tmp_path); + const cleanup = b.addRemoveDirTree(.{ .src_path = .{ .owner = b, .sub_path = tmp_path } }); cleanup.step.dependOn(case_step); step.dependOn(&cleanup.step); @@ -82,7 +82,7 @@ pub fn addCliTests(b: *std.Build, exercises: []const Exercise) *Step { const verify = CheckStep.create(b, exercises, stderr); verify.step.dependOn(&cmd.step); - const cleanup = b.addRemoveDirTree(tmp_path); + const cleanup = b.addRemoveDirTree(.{ .src_path = .{ .owner = b, .sub_path = tmp_path } }); cleanup.step.dependOn(&verify.step); step.dependOn(&cleanup.step);