Wonderful! Thank you, Paul.
Regards,
Lutz
On 15.04.21, 21:02, "Hohensee, Paul" <[email protected]> wrote:
Pushed.
-----Original Message-----
From: jdk-updates-dev <[email protected]> on behalf of
"Hohensee, Paul" <[email protected]>
Date: Thursday, April 15, 2021 at 10:55 AM
To: "Schmidt, Lutz" <[email protected]>,
"[email protected]" <[email protected]>
Cc: build-dev <[email protected]>
Subject: RE: [11u] RFR(M): 8233787 backport: Break cycle in vm_version*
includes
I'll sponsor.
-----Original Message-----
From: "Schmidt, Lutz" <[email protected]>
Date: Thursday, April 15, 2021 at 7:16 AM
To: "Hohensee, Paul" <[email protected]>,
"[email protected]" <[email protected]>
Cc: build-dev <[email protected]>
Subject: RE: [11u] RFR(M): 8233787 backport: Break cycle in vm_version*
includes
Dear all,
would somebody please be willing to sponsor this backport patch?
Thank you,
Lutz
On 14.04.21, 19:05, "jdk-updates-dev on behalf of Schmidt, Lutz"
<[email protected] on behalf of [email protected]> wrote:
Thanks for reviewing, Paul!
Best,
Lutz
On 14.04.21, 19:01, "Hohensee, Paul" <[email protected]> wrote:
Lgtm.
Thanks,
Paul
-----Original Message-----
From: jdk-updates-dev <[email protected]> on
behalf of "Schmidt, Lutz" <[email protected]>
Date: Tuesday, April 13, 2021 at 12:51 AM
To: "[email protected]"
<[email protected]>
Cc: build-dev <[email protected]>
Subject: [11u] RFR(M): 8233787 backport: Break cycle in vm_version*
includes
Dear Community,
I would appreciate receiving reviews for this downport change. It
consists of many modified files. In most cases, it’s only #include statement
changes, caused by factoring out abstract_vm_version.{c|h}pp from
vm_version.{c|h}pp. The change did not apply cleanly, for the most part because
of this split. The other merge conflicts were trivial (include rearrangement
and copyright headers).
Original bug:
https://bugs.openjdk.java.net/browse/JDK-8233787
Downport webrev:
https://cr.openjdk.java.net/~lucy/webrevs/8233787.11u.01/
Merge conflicts:
https://cr.openjdk.java.net/~lucy/webrevs/8233787-jdk11u.conflicts
Conflict resolve diff:
https://cr.openjdk.java.net/~lucy/webrevs/8233787-jdk11u.conflictresolve
Tests:
SAP's internal build and test farm (all OpenJDK platforms (no
32-bit), and more). Tests include JCK, jtreg (hotspot and jdk), and SAP-private
tests. No issues found.
Your effort is very much appreciated!
Thanks,
Lutz
P.S.: build-dev on CC: because a small build change is included.
