Only disable monolith & physics_test_wgpu for Windows

hazeycode · hazeycode · commit 47b9f27d4080 · 2025-01-04T03:04:20.000Z
Working on Linux See #730
diff --git a/build.zig b/build.zig
@@ -141,11 +141,9 @@ pub const samples_cross_platform = struct {
     pub const instanced_pills_wgpu = @import("samples/instanced_pills_wgpu/build.zig");
     pub const layers_wgpu = @import("samples/layers_wgpu/build.zig");
     pub const minimal_zgpu_zgui = @import("samples/minimal_zgpu_zgui/build.zig");
-    // TODO(hazeycode): Get monolith building again. See https://github.com/zig-gamedev/zig-gamedev/pull/718
-    //pub const monolith = @import("samples/monolith/build.zig");
+    pub const monolith = @import("samples/monolith/build.zig");
     pub const physically_based_rendering_wgpu = @import("samples/physically_based_rendering_wgpu/build.zig");
-    // TODO(hazeycode): Get monolith building again. See https://github.com/zig-gamedev/zig-gamedev/pull/718
-    //pub const physics_test_wgpu = @import("samples/physics_test_wgpu/build.zig");
+    pub const physics_test_wgpu = @import("samples/physics_test_wgpu/build.zig");
     pub const procedural_mesh_wgpu = @import("samples/procedural_mesh_wgpu/build.zig");
     pub const textured_quad_wgpu = @import("samples/textured_quad_wgpu/build.zig");
     pub const triangle_wgpu = @import("samples/triangle_wgpu/build.zig");
@@ -176,7 +174,16 @@ pub const samples_web = struct {
 };
 
 fn buildAndInstallSamples(b: *std.Build, options: anytype, comptime samples: anytype) void {
-    inline for (comptime std.meta.declarations(samples)) |d| {
+    buildAndInstallSample: inline for (comptime std.meta.declarations(samples)) |d| {
+        // TODO: Get these samples working on Windows again. See https://github.com/zig-gamedev/zig-gamedev/issues/730
+        if (options.target.query.os_tag == .windows) {
+            inline for (.{ "monolith", "physics_test_wgpu" }) |name| {
+                comptime if (std.mem.eql(u8, name, d.name)) {
+                    continue :buildAndInstallSample;
+                };
+            }
+        }
+
         const exe = @field(samples, d.name).build(b, options);
 
         // TODO: Problems with LTO on Windows.
