Skip to content

Commit

Permalink
build: replace the logo step with the header step
Browse files Browse the repository at this point in the history
Remove the logo step, and use PrintStep for the header step.

The logo step was added as a quick fix after the Builder.addLog function
was removed.

Now the logo is no longer shown when running `zig build -l` or
`zig build -h`.
  • Loading branch information
perillo committed Apr 4, 2023
1 parent 4bd2b62 commit 9354de0
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions build.zig
Original file line number Diff line number Diff line change
Expand Up @@ -533,13 +533,11 @@ pub fn build(b: *Build) !void {
\\
\\
;
const header_step = b.step("info", logo);

const logo_step = PrintStep.create(b, logo, std.io.getStdErr());
logo_step.step.dependOn(header_step);
const header_step = PrintStep.create(b, logo, std.io.getStdErr());

const verify_all = b.step("ziglings", "Check all ziglings");
verify_all.dependOn(header_step);
verify_all.dependOn(&header_step.step);
b.default_step = verify_all;

var prev_chain_verify = verify_all;
Expand Down Expand Up @@ -574,7 +572,7 @@ pub fn build(b: *Build) !void {
chain_verify.dependOn(&verify_step.step);

const named_chain = b.step(b.fmt("{s}_start", .{key}), b.fmt("Check all solutions starting at {s}", .{ex.main_file}));
named_chain.dependOn(header_step);
named_chain.dependOn(&header_step.step);
named_chain.dependOn(chain_verify);

prev_chain_verify.dependOn(chain_verify);
Expand Down

0 comments on commit 9354de0

Please sign in to comment.