PR/59711 CVS commit: src/common/dist/zlib

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

The following reply was made to PR lib/59711; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59711 CVS commit: src/common/dist/zlib
Date: Sun, 19 Oct 2025 21:33:35 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Sun Oct 19 21:33:35 UTC 2025
 
 Modified Files:
 	src/common/dist/zlib: zconf.h
 
 Log Message:
 zlib: Ignore HAVE_UNISTD_H and HAVE_STDARG_H in zconf.h.
 
 This leaves our build of libz.so.1 intact, with no ABI changes --
 z_off_t is still long, internally.  But it avoids exposing a
 different ABI to callers that use autoconf and define HAVE_UNISTD_H,
 so they don't accidentally get z_off_t as off_t.
 
 Later, we can unconditionally take z_off_t to be off_t, to enable
 64-bit file access on LP32 platforms, which we should have done a
 long time ago -- but that requires a shared library major bump (or
 versioning symbols with compat wrappers) because it changes the ABI.
 
 PR lib/59711: "#define HAVE_UNISTD_H 1" breaks 32-bit libz
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.5 -r1.6 src/common/dist/zlib/zconf.h
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.