Merge pull request #1 from allyourcodebase/static

Static build

Author: MasonRemaley

README.md

@@ -2,12 +2,11 @@
 
 Pipewire client library, ported to the Zig build system.
 
+
 ## Motivation
 
-I want a static executable that can play audio and turn screen contents into a video feed.
+I want a static executable that can play audio and turn screen contents into a video feed. The pipewire library makes heavy use of `dlopen` internally, so this is nontrivial.
 
 ## Status
 
-I got the screen-play example that displays the current webcam feed compiling and running (see `zig build screen-play`.)
-
-The pipewire library makes heavy use of `dlopen` internally, so further work will be needed to link statically to it.
+You can run the `video-play` example with `zig build video-play` to see the current webcam feed. This currently works without pipewire accessing the dynamic linker, but the example executable uses SDL so it still has access to it. I plan to port the example away from SDL so that this can be changed.

build.zig

@@ -20,6 +20,10 @@
             .url = "git+https://github.com/allyourcodebase/SDL3#af9c32ce4824e0ad4337447434c40f2b672faf94",
             .hash = "sdl-0.0.0-i4QD0btgqAABajEXrQnyZr1xVsk7LM48w2nBmuQ5gdfr",
         },
+        .zin = .{
+            .url = "git+https://github.com/marler8997/zin#d3e230690f09debdb99dffe4586ad19f9dfb4716",
+            .hash = "zin-0.0.0-W7QDx6kKAwCVg-wWMO4FdpvZMH66NJpc9wFDdxp5c8E2",
+        },
     },
     .paths = .{
         "LICENSE",

build.zig.zon

@@ -1,11 +1,11 @@
+//! Generates the pipewire client config. We can almost use `addConfigHeader` for this with the
+//! `autoconf_at` style, but not quite as it adds a c style comment to the first line (explaining
+//! that the file is generated) which isn't allowed by this syntax.
+
 const std = @import("std");
 const options = @import("options");
-
 const assert = std.debug.assert;
 
-/// Generate the pipewire config. We can almost use `addConfigHeader` for this with the
-/// `autoconf_at` style, but not quite as it adds a c style comment to the first line (explaining
-/// that the file is generated) which isn't allowed by this syntax.
 pub fn main() !void {
     var gpa = std.heap.GeneralPurposeAllocator(.{ .thread_safe = false }){};
     defer if (gpa.deinit() != .ok) @panic("leak detected");
@@ -32,6 +32,17 @@ pub fn main() !void {
     var output_buf: [4096]u8 = undefined;
     var writer = output.writerStreaming(&output_buf);
 
+    // Ideally we'd just use embed file rather than emit this warning but I'm having trouble
+    // getting `addEmbedPath` to work, so this will do for now. Not sure if this is a bug or user
+    // error.
+    try writer.interface.writeAll("# ");
+    try writer.interface.splatByteAll('#', 80 - 2);
+    try writer.interface.writeAll("\n");
+    try writer.interface.writeAll("# Generated by Zig's build system. Do not edit.\n");
+    try writer.interface.writeAll("# ");
+    try writer.interface.splatByteAll('#', 80 - 2);
+    try writer.interface.writeAll("\n\n");
+
     while (true) {
         _ = reader.interface.streamDelimiter(&writer.interface, '@') catch |err| switch (err) {
             error.EndOfStream => break,