Ensure that the Kotlin compiler adds method param names to bytecode

We already do this for Java, so let's make it consistent for Kotlin too Fixes: quarkusio#6041
gsmet · Jan 2, 2020 · aea705d · aea705d
1 parent 8c42ac5
commit aea705d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/...-descriptor-json/src/main/resources/templates/basic-rest/kotlin/build.gradle-template.ftl b/...-descriptor-json/src/main/resources/templates/basic-rest/kotlin/build.gradle-template.ftl
@@ -42,6 +42,7 @@ java {
 
 compileKotlin {
     kotlinOptions.jvmTarget = JavaVersion.VERSION_1_8
+    kotlinOptions.javaParameters = true
 }
 
 compileTestKotlin {

diff --git a/...tform-descriptor-json/src/main/resources/templates/basic-rest/kotlin/pom.xml-template.ftl b/...tform-descriptor-json/src/main/resources/templates/basic-rest/kotlin/pom.xml-template.ftl
@@ -106,6 +106,7 @@
                     </execution>
                 </executions>
                 <configuration>
+                    <javaParameters>true</javaParameters>
                     <!-- Soon to be replaced by plugin that will pre-configure all necessary annotations -->
                     <compilerPlugins>
                         <plugin>all-open</plugin>

diff --git a/...tlin/deployment/src/main/java/io/quarkus/kotlin/deployment/KotlinCompilationProvider.java b/...tlin/deployment/src/main/java/io/quarkus/kotlin/deployment/KotlinCompilationProvider.java
@@ -37,6 +37,7 @@ public Set<String> handledExtensions() {
     @Override
     public void compile(Set<File> filesToCompile, Context context) {
         K2JVMCompilerArguments compilerArguments = new K2JVMCompilerArguments();
+        compilerArguments.setJavaParameters(true);
         if (context.getCompilePluginArtifacts() != null && !context.getCompilePluginArtifacts().isEmpty()) {
             compilerArguments.setPluginClasspaths(context.getCompilePluginArtifacts().toArray(new String[0]));
         }