RFR: 8378631: Update Zlib Data Compression Library to Version 1.3.2
Jaikiran Pai
jpai at openjdk.org
Wed Feb 25 13:50:14 UTC 2026
On Wed, 25 Feb 2026 13:28:03 GMT, Alan Bateman <alanb at openjdk.org> wrote:
> @jaikiran No changes from upstream, right? (ignoring the make file update to compile with -DHAVE_STDARG_H=1).
That's correct - no changes compared to upstream other than the make file change. The trivial make file changes, in our JDK build, were necessitated due to a change in the upstream code which now relies on these macro values to be 1 for it to include certain functionality from zlib https://github.com/madler/zlib/commit/7108497fda9d4536a1afade7f42266e06dca4488
> Also would be possible to update the PR description to say what testing has been done as the defaults only use the in-tree version on Windows.
I was planning to add that detail when I opened the PR, but I hesitated a bit. I have tested tier1, tier2, tier3 on all platforms in our CI and they all passed fine. But that testing was done a few days back, so I didn't include that detail and instead triggered a new run. I should have the results soon and I'll update here once I have them.
> as the defaults only use the in-tree version on Windows.
The in-tree version also gets used on macosx-aarch64. We were forced to do that because of an extremely hard to debug intermittent failures in zlib that was shipped in macos. Details in https://bugs.openjdk.org/browse/JDK-8286623. So right now, even macosx-aarch64 uses bundled https://github.com/openjdk/jdk/blob/master/make/autoconf/lib-bundled.m4#L219. The original issue that we reported to Apple https://bugs.openjdk.org/browse/JDK-8282954 has reportedly been addressed in macosx 13.x and higher, but we haven't yet decided if we want to go back to that using "system" on that platform.
I also plan to run higher tier tests before integrating.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/29916#issuecomment-3959366274
More information about the core-libs-dev
mailing list