RFR: 8295530: Update Zlib Data Compression Library to Version 1.2.13
Lance Andersen
lancea at openjdk.org
Thu Oct 20 10:17:50 UTC 2022
On Thu, 20 Oct 2022 02:02:40 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Hi all,
>>
>> Please review this PR which will update the Zlib Data Compression Library from 1.2.11 to 1.2.13.
>>
>> - I have run all of the Mach5 tiers which did not show any issues related to the upgrade.
>> - The JCK zip/jar tests also continue to pass.
>> - Jai ran his macOS aarch64 test(for [JDK-8282954](https://bugs.openjdk.org/browse/JDK-8282954)) with the zlib 1.2.13 port and has not seen the failure as seen with the MacOS aarch64 native zlib port
>>
>>
>> Best,
>> Lance
>
> src/java.base/share/native/libzip/zlib/README line 6:
>
>> 4: thread safe. The data format used by the zlib library is described by RFCs
>> 5: (Request for Comments) 1950 to 1952 in the files
>> 6: http://tools.ietf.org/html/rfc1950 (zlib format), rfc1951 (deflate format) and
>
> Hello Lance, it appears that we had done this change to this file in the JDK as part of https://bugs.openjdk.org/browse/JDK-8283525. As Sean notes in that issue, the `http` link is back to working but `https` was still prefered during that change. Should we stick with `https`?
Hi Jai,
Given this is how it is in the README in the upstream repository and it is accessible, I would prefer not to change it and when/if we do, we should have the change made upstream
-------------
PR: https://git.openjdk.org/jdk/pull/10773
More information about the core-libs-dev
mailing list