maven-plugin: fix kotlin compiler args parameter setting (#339)

* maven-plugin: fix kotlin compiler args parameter setting * test: add kotlin compiler arg to validate it works
casid · Apr 5, 2024 · 376f241 · 376f241
1 parent e8fca09
commit 376f241
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/jte-maven-plugin/src/main/java/gg/jte/maven/CompilerMojo.java b/jte-maven-plugin/src/main/java/gg/jte/maven/CompilerMojo.java
@@ -137,7 +137,7 @@ public void execute() {
         templateEngine.setHtmlCommentsPreserved(htmlCommentsPreserved);
         templateEngine.setBinaryStaticContent(binaryStaticContent);
         templateEngine.setCompileArgs(calculateCompileArgs());
-        templateEngine.setCompileArgs(kotlinCompileArgs);
+        templateEngine.setKotlinCompileArgs(kotlinCompileArgs);
 
         int amount;
         try {

diff --git a/test/jte-runtime-test-kotlin/pom.xml b/test/jte-runtime-test-kotlin/pom.xml
@@ -14,7 +14,7 @@
         <maven.deploy.skip>true</maven.deploy.skip>
         <skipNexusStagingDeployMojo>true</skipNexusStagingDeployMojo>
         <jacoco.version>0.8.10</jacoco.version>
-        <kotlin.version>1.4.30</kotlin.version>
+        <kotlin.version>1.9.10</kotlin.version>
     </properties>
 
     <dependencies>
@@ -96,6 +96,10 @@
                     <sourceDirectory>${basedir}/src/main/jte</sourceDirectory>
                     <targetDirectory>${basedir}/jte-classes</targetDirectory>
                     <contentType>Html</contentType>
+                    <kotlinCompileArgs>
+                        <kotlinCompileArg>-jvm-target</kotlinCompileArg>
+                        <kotlinCompileArg>${java.version}</kotlinCompileArg>
+                    </kotlinCompileArgs>
                 </configuration>
                 <executions>
                     <execution>