[8u] RFR: 8244154: Update SunPKCS11 provider with PKCS11 v3.0 header files
Andrew Hughes
gnu.andrew at redhat.com
Thu Jul 29 16:24:45 UTC 2021
On 14:29 Thu 29 Jul , Jonathan Dowland wrote:
> Hi there,
>
> Please consider this backport to jdk8u for parity with Oracle 8u291.
>
> The unshuffled 11u patch does not apply cleanly to 8u:
>
> * License file pkcs11cryptotoken.md does not exist in 8u. pre-Jigsaw
> it's bundled into THIRD_PARTY_README. The below webrev URI includes
> updates to THIRD_PARTY_README for all 8 jdk8u repositories.
>
> * 11u patch fails to apply 1 out of 52 hunks to pkcs11t.h: this is
> partly due to out-of-order backporting (8265462 brought forward
> one line of changes from this patch). The failed hunk was largely
> (but not entirely) whitespace changes and the introduction of brand
> new code.
>
> After resolving it by hand, the result is identical to the version of
> the file in jdk11u.
>
> Tests: I ran jdk_tier1 before and after and got the same results
> (1135 passed, 196 failed, 10 not run). I'd appreciate any advice on good
> tests to run to exercise these changes.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8244154
> webrev: https://cr.openjdk.java.net/~jdowland/webrevs/JDK-8244154/webrev.00/
> zipped webrev: https://cr.openjdk.java.net/~jdowland/webrevs/JDK-8244154/webrev.00.zip
>
>
> Thanks,
>
> --
> Jonathan Dowland <jdowland at redhat.com>
> Senior Software Engineer, OpenJDK, Red Hat
>
This looks fine to me. I only looked at the jdk patch; I assume the others are
just the THIRD_PARTY_README hunk applied to the other repos?
If so, please flag for approval.
Thanks,
--
Andrew :)
Pronouns: he / him or they / them
Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)
PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222
More information about the jdk8u-dev
mailing list