Skip to content
This repository has been archived by the owner on Feb 4, 2025. It is now read-only.

Commit

Permalink
remove unused run step, doesn't run yet in latest zig ziglang/zig#2041
Browse files Browse the repository at this point in the history 
this was never actually used anyway
  • Loading branch information
@Akuli
Akuli committed Mar 10, 2019
1 parent 9826500 commit aac901d
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions asdar/build.zig
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,6 @@ pub fn build(b: *Builder) void {
exe.linkSystemLibrary("c");
exe.linkSystemLibrary("gmp");

const run_step = b.step("run", "Run the app");
const run_cmd = b.addCommand(".", b.env_map, [][]const u8{exe.getOutputPath()});
run_step.dependOn(&run_cmd.step);
run_cmd.step.dependOn(&exe.step);

b.default_step.dependOn(&exe.step);
b.installArtifact(exe);
}

2 comments on commit aac901d

@andrewrk
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For what it's worth you can fix it like this:

const run_step = b.step("run", "Run the app");
const run_cmd = exe.run();
run_step.dependOn(&run_cmd.step);

No longer necessary: run_cmd.step.dependOn(&exe.step);

@Akuli
Copy link
Owner Author

@Akuli Akuli commented on aac901d Mar 10, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't need that

Please sign in to comment.