[RFR] [11u] 11.0.9+11 / 11.0.9-ga

[Langer, Christoph]

Hi,

to me it looks as if the webrev was generated on a branch that contains other local changes, e.g. in make/lib/Lib-jdk.crypto.ec.gmk as well. So, those of course mustn't be bushed.

I guess the correct patchset is contained in https://cr.openjdk.java.net/~andrew/openjdk11/11.0.9/jdk11.changeset, right?

Thanks
Christoph

> -----Original Message-----
> From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> On
> Behalf Of Aleksey Shipilev
> Sent: Mittwoch, 21. Oktober 2020 08:41
> To: Andrew Hughes <gnu.andrew at redhat.com>; jdk-updates-
> dev at openjdk.java.net
> Subject: Re: [RFR] [11u] 11.0.9+11 / 11.0.9-ga
> 
> On 10/21/20 7:59 AM, Andrew Hughes wrote:
> > Here are the remaining changes for the jdk11u repository:
> >
> > Webrev: https://cr.openjdk.java.net/~andrew/openjdk11/11.0.9/
> 
> *) What is the changes in:
> 
> src/java.base/share/classes/sun/security/ssl/SupportedGroupsExtension.jav
> a
>    src/java.base/share/classes/sun/security/util/CurveDB.java
>    src/java.base/share/conf/security/java.security
> 
> src/java.xml.crypto/share/classes/com/sun/org/apache/xml/internal/securit
> y/algorithms/implementations/ECDSAUtils.java
>    src/jdk.crypto.ec/share/native/libsunec/impl/ecl-curve.h
>    src/jdk.crypto.ec/share/native/libsunec/impl/ecl.c
>    src/jdk.crypto.ec/share/native/libsunec/impl/oid.c
>    test/jdk/sun/security/ec/TestEC.java
>    test/jdk/sun/security/pkcs11/sslecc/ClientJSSEServerJSSE.java
> 
>   They are not recorded to belong to any particular changeset. And as far as I
> can see, there are no
> symmetric change in 8u CPU.
> 
> 
> *) As well as in 8u CPU,
>      src/share/native/sun/awt/image/jpeg/jdhuff.c
>      src/share/native/sun/awt/image/jpeg/jdinput.c
>      src/share/native/sun/awt/image/jpeg/jdmarker.c
>      src/share/native/sun/awt/image/jpeg/jpeglib.h
> 
>    ...seem weirdly empty. They are intended to be that way, right?
> 
> 
> > Ok to push?
> 
> Well, I am concerned about seemingly unaccounted files above. Please verify
> what is up with those first?
> 
> --
> Thanks,
> -Aleksey