Use own lookupMethod to avoid dependency on internal GraalVM API

quarkusio · Sep 2, 2022 · df10660 · df10660
1 parent b9ba708
commit df10660
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 2 deletions.
diff --git a/core/deployment/pom.xml b/core/deployment/pom.xml
@@ -194,6 +194,10 @@
                             <version>${project.version}</version>
                         </path>
                     </annotationProcessorPaths>
+                    <compilerArgs>
+                        <arg>--add-exports</arg>
+                        <arg>java.base/jdk.internal.module=ALL-UNNAMED</arg>
+                    </compilerArgs>
                 </configuration>
             </plugin>
         </plugins>

diff --git a/core/deployment/src/main/java/io/quarkus/deployment/steps/NativeImageFeatureStep.java b/core/deployment/src/main/java/io/quarkus/deployment/steps/NativeImageFeatureStep.java
@@ -53,6 +53,7 @@
 import io.quarkus.runtime.ResourceHelper;
 import io.quarkus.runtime.graal.ResourcesFeature;
 import io.quarkus.runtime.graal.WeakReflection;
+import jdk.internal.module.Modules;
 
 public class NativeImageFeatureStep {
 
@@ -82,7 +83,7 @@ public class NativeImageFeatureStep {
             String.class);
 
     private static final MethodDescriptor LOOKUP_METHOD = ofMethod(
-            "com.oracle.svm.util.ReflectionUtil",
+            NativeImageFeatureStep.class,
             "lookupMethod", Method.class, Class.class, String.class, Class[].class);
     private static final MethodDescriptor FOR_NAME = ofMethod(
             Class.class, "forName", Class.class, String.class, boolean.class, ClassLoader.class);
@@ -688,4 +689,18 @@ private ReflectionInfo(boolean constructors, boolean methods, boolean fields, bo
         }
     }
 
+    private static Method lookupMethod(Class<?> declaringClass, String methodName, Class<?>... parameterTypes)
+            throws NoSuchMethodException {
+        Method result = declaringClass.getDeclaredMethod(methodName, parameterTypes);
+        Module declaringModule = declaringClass.getModule();
+        String packageName = declaringClass.getPackageName();
+
+        if (declaringModule.isOpen(packageName)) {
+            Modules.addOpensToAllUnnamed(declaringModule, packageName);
+        }
+
+        result.setAccessible(true);
+        return result;
+    }
+
 }
diff --git a/pom.xml b/pom.xml
@@ -53,7 +53,6 @@
     <properties>
         <maven.compiler.target>11</maven.compiler.target>
         <maven.compiler.source>11</maven.compiler.source>
-        <maven.compiler.release>11</maven.compiler.release>
         <maven.compiler.parameters>true</maven.compiler.parameters>
 
         <graalvmHome>${env.GRAALVM_HOME}</graalvmHome>