Merge pull request #6310 from geoand/#6041

Ensure that the Kotlin compiler adds method param names to bytecode
quarkusio · Dec 22, 2019 · 868aa27 · 868aa27
2 parents ac1ed45 + 8e06dad
commit 868aa27
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]));
         }