RFR [XS] : 8235478: [jdk11] handle VS2017 15.9 and VS2019 in vm_version.cpp
Langer, Christoph
christoph.langer at sap.com
Thu Dec 12 10:46:36 UTC 2019
Hi Matthias,
the change itself looks good.
However, your approach to create a new bug which is actually a backport of two other bugs is not the right thing to do. I converted JDK-8235478 to be a backport of JDK-8235243. Now, please request backport approval in JDK-8235243 and JDK-8235325. Once it's approved, please push your change with a commit message that resolves both items, that is:
8235243: handle VS2017 15.9 and VS2019 in abstract_vm_version
8235325: build failure on Linux after 8235243
Reviewed-by: dholmes, mdoerr
This will pick up and resolve JDK-8235478 as backport of 8235243 and create/resolve another backport item for 8235325.
Best regards
Christoph
> -----Original Message-----
> From: hotspot-dev <hotspot-dev-bounces at openjdk.java.net> On Behalf Of
> Baesken, Matthias
> Sent: Montag, 9. Dezember 2019 12:37
> To: jdk-updates-dev at openjdk.java.net
> Cc: 'hotspot-dev at openjdk.java.net' <hotspot-dev at openjdk.java.net>
> Subject: RFR [XS] : 8235478: [jdk11] handle VS2017 15.9 and VS2019 in
> vm_version.cpp
>
> Hello, please review this small jdk11 change .
> It handles VS2017 15.9 and VS2019 in vm_version.cpp .
>
> More or less it is a jdk11 - downport of what 8235243 + 8235325 do , but in
> jdk/jdk .
>
> Bug/webrev :
>
> https://bugs.openjdk.java.net/browse/JDK-8235478
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8235478.0/
>
>
> Thanks, Matthias
More information about the jdk-updates-dev
mailing list