ibmruntimes / openj9-openjdk-jdk21

GNU General Public License v2.0
8 stars 40 forks source link

Fix -Djdk.reflect.useDirectMethodHandle=false #195

Closed babsingh closed 2 months ago

babsingh commented 2 months ago

See https://github.com/eclipse-openj9/openj9/issues/20155 for more details on the failure and the fix.

Fixes: https://github.com/eclipse-openj9/openj9/issues/20155

babsingh commented 2 months ago

@gacholio Can you review this change?

fyi @r30shah @tajila

Note: -Djdk.reflect.useDirectMethodHandle=false was introduced in JDK18+ as part of JEP416, and it is no longer supported in JDK22+: https://bugs.openjdk.org/browse/JDK-8309635. -Djdk.reflect.useDirectMethodHandle=false is only supported between JDK18 to JDK21. So, the fix is only needed for JDK21 since we no longer support JDKs 18-20.

gacholio commented 2 months ago

jenkins line endings check

gacholio commented 2 months ago

jenkins test sanity zlinux jdk8

babsingh commented 2 months ago

jdk8 build will probably fail with jdk21 extension repo. To be safe, relaunching the build:

jenkins test sanity zlinux jdk21

babsingh commented 2 months ago

Github GUI is not updating. Here is the latest PR build: https://openj9-jenkins.osuosl.org/job/PullRequest-OpenJDK21/82.

gacholio commented 2 months ago

jdk8 build will probably fail with jdk21 extension repo. To be safe, relaunching the build:

Right, didn't read closely enough.

gacholio commented 2 months ago

Failure looks infra-related. Do builds even work in this repo?

babsingh commented 2 months ago

The build worked. There are only two identical failures, but they occur in the GC and are unrelated to this change. The below failures are known and documented in https://github.com/eclipse-openj9/openj9/issues/20150.

gacholio commented 2 months ago

I do not have permission to merge this.

babsingh commented 2 months ago

++ @AdamBrousseau

babsingh commented 2 months ago

@gacholio You should be able to merge now. @AdamBrousseau has granted you the needed permissions.