Wireshark mailing list archives
Re: Making zlib a required dependency
From: Guy Harris <guy () alum mit edu>
Date: Sat, 11 Nov 2017 10:42:10 -0800
On Nov 11, 2017, at 7:04 AM, João Valverde <joao.valverde () tecnico ulisboa pt> wrote:
I pushed a change making zlib 1.2.3 or greater a required build dependency: https://code.wireshark.org/review/#/c/24352/1
Does that change also assume that pkg-config will find zlib? If so, that's an inappropriate assumption on the desktop UN*X with the largest market share: $ pkg-config --list-all | egrep -i 'zlib|libz' $ ls /usr/lib/libz.* /usr/lib/libz.1.1.3.dylib /usr/lib/libz.1.dylib /usr/lib/libz.1.2.5.dylib /usr/lib/libz.dylib /usr/lib/libz.1.2.8.dylib $ ls /usr/include/zlib.h /usr/include/zlib.h $ which pkg-config /usr/local/bin/pkg-config $ sw_vers ProductName: Mac OS X ProductVersion: 10.12.6 BuildVersion: 16G1036 macOS 1) has a bundled zlib, and includes header files for it in Xcode and the command-line build tools; 2) does *not* have a bundled pkg-config, and thus does *not* provide .pc files for zlib; so anything that searches for zlib using pkg-config and that does *NOT* fall back on using an old-fashioned mechanism to find it if pkg-config doesn't find t will fail. ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Making zlib a required dependency João Valverde (Nov 11)
- Re: Making zlib a required dependency Pascal Quantin (Nov 11)
- Re: Making zlib a required dependency João Valverde (Nov 11)
- Re: Making zlib a required dependency Guy Harris (Nov 11)
- Re: Making zlib a required dependency João Valverde (Nov 11)
- Re: Making zlib a required dependency Guy Harris (Nov 11)
- Re: Making zlib a required dependency João Valverde (Nov 14)
- Re: Making zlib a required dependency Pascal Quantin (Nov 11)