CVS commit: pkgsrc/devel/zlib

["Benny Siegert" <bsiegert%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   bsiegert
Date:           Tue Jun  4 18:33:45 UTC 2024

Modified Files:
        pkgsrc/devel/zlib: Makefile

Log Message:
zlib: always include unistd.h on Linux

Newer compilers, such as gcc14 in Fedora 40, make calling undeclared
functions an error. zlib uses lseek, which is in unistd.h. The existing
zconf.h has complex logic to include unistd.h in some cases, when it should
probably just include it unconditionally.

Set -DZ_HAVE_UNISTD_H in CFLAGS.Linux. This should probably just be added
unconditionally, but I was hesitant to do so without asking.


To generate a diff of this commit:
cvs rdiff -u -r1.63 -r1.64 pkgsrc/devel/zlib/Makefile

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/devel/zlib/Makefile
diff -u pkgsrc/devel/zlib/Makefile:1.63 pkgsrc/devel/zlib/Makefile:1.64
--- pkgsrc/devel/zlib/Makefile:1.63     Thu Feb  8 21:33:34 2024
+++ pkgsrc/devel/zlib/Makefile  Tue Jun  4 18:33:45 2024
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.63 2024/02/08 21:33:34 adam Exp $
+# $NetBSD: Makefile,v 1.64 2024/06/04 18:33:45 bsiegert Exp $
 
 DISTNAME=      zlib-1.3.1
 CATEGORIES=    devel
@@ -26,6 +26,7 @@ CPPFLAGS.SunOS+=      -DNO_DIVIDE
 .endif
 LDFLAGS.SunOS+=        -Wl,-zignore    # avoid unused reference to libgcc_s.so
 
+CFLAGS.Linux+= -DZ_HAVE_UNISTD_H
 LDFLAGS.Linux+=        -Wl,--version-script,zlib.map
 
 SUBST_CLASSES+=                pc