ReviewOS

also looking at this

craft-native/craft

fix(release): stop shipping the wrong binaries, and delete the dead notifier

#43
Merged glennmichael123 wants to merge fix/release-artifact-hygiene into main
2 files +24 -28
packages/zig/build.zigmodified+6-2
Changes to packages/zig/build.zig
@@ -82,10 +82,14 @@ pub fn build(b: *std.Build) void {
8282 // Add system libraries based on platform
8383 const target_os = target.result.os.tag;
8484 linkPlatformLibraries(b, exe.root_module, target_os, macos_sdk);
85 b.installArtifact(exe);
8685
86 // Deliberately not installed. `zig-out/bin` is what the release packager
87 // zips, whole and unfiltered, so anything installed here ships to users —
88 // and the demo was riding along in every archive, unsigned on macOS next
89 // to a notarized `craft`, because the signing loop names only `craft`.
90 // `zig build run-demo` still builds and runs it; it just does not end up
91 // in the install prefix.
8792 const run_cmd = b.addRunArtifact(exe);
88 run_cmd.step.dependOn(b.getInstallStep());
8993
9094 const run_step = b.step("run-demo", "Run the demo app");
9195 run_step.dependOn(&run_cmd.step);