Skip to content

Commit

Permalink
Bug fix: added a call to in build.zig
Browse files Browse the repository at this point in the history
  • Loading branch information
Durobot committed Mar 4, 2024
1 parent 9ced743 commit d5a97d7
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 79 deletions.
68 changes: 15 additions & 53 deletions build.zig
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,20 @@ pub fn build(b: *std.Build) void
// set a preferred release mode, allowing the user to decide how to optimize.
const optimize = b.standardOptimizeOption(.{});

// const mod = b.addModule("zigStructPrint", // package_name
_ = b.addModule("zigStructPrint", // package_name
.{
.root_source_file = .{ .path = "src/zsp.zig" },
.target = target,
.optimize = optimize,
});

const lib = b.addStaticLibrary(
.{
.name = "zigStructPrint",
// In this case the main source file is merely a path, however, in more
// complicated build scripts, this could be a generated file.
.root_source_file = .{ .path = "src/root.zig" },
.root_source_file = .{ .path = "src/zsp.zig" },
.target = target,
.optimize = optimize,
});
Expand All @@ -31,66 +39,20 @@ pub fn build(b: *std.Build) void
// running `zig build`).
b.installArtifact(lib);

// const exe = b.addExecutable(
// .{
// .name = "zigStructPrint",
// .root_source_file = .{ .path = "src/main.zig" },
// .target = target,
// .optimize = optimize,
// });

// This declares intent for the executable to be installed into the
// standard location when the user invokes the "install" step (the default
// step when running `zig build`).
// b.installArtifact(exe);

// This *creates* a Run step in the build graph, to be executed when another
// step is evaluated that depends on it. The next line below will establish
// such a dependency.
// const run_cmd = b.addRunArtifact(exe);

// By making the run step depend on the install step, it will be run from the
// installation directory rather than directly from within the cache directory.
// This is not necessary, however, if the application depends on other installed
// files, this ensures they will be present and in the expected location.
// run_cmd.step.dependOn(b.getInstallStep());

// This allows the user to pass arguments to the application in the build
// command itself, like this: `zig build run -- arg1 arg2 etc`
// if (b.args) |args| {
// run_cmd.addArgs(args);
// }

// This creates a build step. It will be visible in the `zig build --help` menu,
// and can be selected like this: `zig build run`
// This will evaluate the `run` step rather than the default, which is "install".
// const run_step = b.step("run", "Run the app");
// run_step.dependOn(&run_cmd.step);

// Creates a step for unit testing. This only builds the test executable
// but does not run it.
const lib_unit_tests = b.addTest(
.{
.root_source_file = .{ .path = "src/root.zig" },
.target = target,
.optimize = optimize,
});

const run_lib_unit_tests = b.addRunArtifact(lib_unit_tests);

// const exe_unit_tests = b.addTest(
// const lib_unit_tests = b.addTest(
// .{
// .root_source_file = .{ .path = "src/main.zig" },
// .root_source_file = .{ .path = "src/tests.zig" },
// .target = target,
// .optimize = optimize,
// });

// const run_exe_unit_tests = b.addRunArtifact(exe_unit_tests);
//
// const run_lib_unit_tests = b.addRunArtifact(lib_unit_tests);

// Similar to creating the run step earlier, this exposes a `test` step to
// the `zig build --help` menu, providing a way for the user to request
// running the unit tests.
const test_step = b.step("test", "Run unit tests");
test_step.dependOn(&run_lib_unit_tests.step);
// test_step.dependOn(&run_exe_unit_tests.step);
// const test_step = b.step("test", "Run unit tests");
// test_step.dependOn(&run_lib_unit_tests.step);
}
27 changes: 1 addition & 26 deletions build.zig.zon
Original file line number Diff line number Diff line change
Expand Up @@ -14,32 +14,7 @@
// `zig build --fetch` can be used to fetch all dependencies of a package, recursively.
// Once all dependencies are fetched, `zig build` no longer requires
// internet connectivity.
.dependencies =
.{
// See `zig fetch --save <url>` for a command-line interface for adding dependencies.
//.example = .{
// // When updating this field to a new URL, be sure to delete the corresponding
// // `hash`, otherwise you are communicating that you expect to find the old hash at
// // the new URL.
// .url = "https://example.com/foo.tar.gz",
//
// // This is computed from the file contents of the directory of files that is
// // obtained after fetching `url` and applying the inclusion rules given by
// // `paths`.
// //
// // This field is the source of truth; packages do not come from a `url`; they
// // come from a `hash`. `url` is just one of many possible mirrors for how to
// // obtain a package matching this `hash`.
// //
// // Uses the [multihash](https://multiformats.io/multihash/) format.
// .hash = "...",
//
// // When this is provided, the package is found in a directory relative to the
// // build root. In this case the package's hash is irrelevant and therefore not
// // computed. This field and `url` are mutually exclusive.
// .path = "foo",
//},
},
.dependencies = .{},

// Specifies the set of files and directories that are included in this package.
// Only files and directories listed here are included in the `hash` that
Expand Down
File renamed without changes.

0 comments on commit d5a97d7

Please sign in to comment.