Inflate MuPDF's streams with the zlib FreeType links

MuPDF compiled its bundled zlib 1.3.1 (ZLIB_SRC) into libmupdf.a, and the
freetype package links the zlib package, 1.3.2, for its gzip module: 68
symbols defined twice, with the linker keeping one of each. MuPDF now links
that same zlib artifact and stops compiling its own. build.zig asks
FreeType's build for zlib with the options FreeType asks with, which the
build's dependency cache answers with the very artifact FreeType links, so
the tty and the gui each carry exactly one zlib.

MuPDF hands zlib its own allocation functions on every stream, so nothing
about where the memory comes from changes. Every page of the three PDFs in
docs/ still renders bit for bit as before, at 144 and at 300 dpi.

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>

1 files changed, 6 insertions, 7 deletions

diff --git a/mupdf.zig b/mupdf.zig
index cda9a38a..a157e850 100644
--- a/mupdf.zig
+++ b/mupdf.zig
@@ -41,6 +41,10 @@ pub fn add(b: *std.Build, io: std.Io, dependency: *std.Build.Dependency, options
     /// ran the slim one. MuPDF asks nothing of FreeType the full
     /// configuration lacks.
     freetype: *std.Build.Step.Compile,
+    /// The zlib FreeType links, which MuPDF links instead of compiling the
+    /// one it bundles (ZLIB_SRC): the two define the same symbols, as the
+    /// FreeTypes do.
+    zlib: *std.Build.Step.Compile,
 }) Result {
     const module = b.createModule(.{
         .target = options.target,
@@ -53,8 +57,9 @@ pub fn add(b: *std.Build, io: std.Io, dependency: *std.Build.Dependency, options
     // debug-locking/assertion paths. linkTo applies the same public-header
     // contract to consumers; keeping it here also covers every archive source.
     module.addCMacro("NDEBUG", "1");
-    // The build's own FreeType, headers and all; see `freetype` above.
+    // The build's own FreeType and zlib, headers and all; see above.
     module.linkLibrary(options.freetype);
+    module.linkLibrary(options.zlib);
 
     // MuPDF itself uses wildcards for these two directories. The dependency
     // is immutable and content-addressed, so discovering the flat C lists at
@@ -97,11 +102,6 @@ pub fn add(b: *std.Build, io: std.Io, dependency: *std.Build.Dependency, options
     });
     module.addCSourceFiles(.{
         .root = dependency.path(""),
-        .files = makeSources(b, makelists, "ZLIB_SRC"),
-        .flags = &.{ "-std=gnu11", "-DHAVE_UNISTD_H", "-DHAVE_STDARG_H" },
-    });
-    module.addCSourceFiles(.{
-        .root = dependency.path(""),
         .files = makeSources(b, makelists, "JBIG2DEC_SRC"),
         .flags = &.{
             "-std=gnu11",
@@ -252,7 +252,6 @@ fn configureModule(module: *std.Build.Module, dependency: *std.Build.Dependency,
     module.addIncludePath(dependency.path("source/pdf"));
     module.addIncludePath(dependency.path("thirdparty/libjpeg"));
     module.addIncludePath(dependency.path("scripts/libjpeg"));
-    module.addIncludePath(dependency.path("thirdparty/zlib"));
     module.addIncludePath(dependency.path("thirdparty/jbig2dec"));
     // encode-jpx.c includes this header before its FZ_ENABLE_JPX guard.
     module.addIncludePath(dependency.path("thirdparty/openjpeg/src/lib/openjp2"));