Merge pull request #264 from bmarwell/support_jdk21

Support JDK21
groovy · Sep 25, 2023 · 5093cfc · 5093cfc
2 parents c70b3cb + 33606ea
commit 5093cfc
Show file tree

Hide file tree

Showing 4 changed files with 36 additions and 2 deletions.
diff --git a/src/main/java/org/codehaus/gmavenplus/mojo/AbstractCompileMojo.java b/src/main/java/org/codehaus/gmavenplus/mojo/AbstractCompileMojo.java
@@ -44,6 +44,11 @@
  */
 public abstract class AbstractCompileMojo extends AbstractGroovySourcesMojo {
 
+    /**
+     * Groovy 4.0.11 version.
+     */
+    protected static final Version GROOVY_4_0_11 = new Version(4, 0, 11);
+
     /**
      * Groovy 4.0.6 version.
      */
@@ -202,6 +207,7 @@ public abstract class AbstractCompileMojo extends AbstractGroovySourcesMojo {
      * Using 18 requires Groovy &gt; 4.0.0-beta-1.
      * Using 19 requires Groovy &gt; 4.0.2.
      * Using 20 requires Groovy &gt; 4.0.6.
+     * Using 21 requires Groovy &gt; 4.0.11.
      */
     @Parameter(property = "maven.compiler.target", defaultValue = "1.8")
     protected String targetBytecode;
@@ -490,7 +496,11 @@ protected Object setupCompilerConfiguration(final File compileOutputDirectory, f
      * org.codehaus.groovy.classgen.asm.WriterController.
      */
     protected void verifyGroovyVersionSupportsTargetBytecode() {
-        if ("20".equals(targetBytecode)) {
+        if ("21".equals(targetBytecode)) {
+            if (groovyOlderThan(GROOVY_4_0_11)) {
+                throw new IllegalArgumentException("Target bytecode " + targetBytecode + " requires Groovy " + GROOVY_4_0_11 + " or newer.");
+            }
+        } else if ("20".equals(targetBytecode)) {
             if (groovyOlderThan(GROOVY_4_0_6)) {
                 throw new IllegalArgumentException("Target bytecode " + targetBytecode + " requires Groovy " + GROOVY_4_0_6 + " or newer.");
             }

diff --git a/src/main/java/org/codehaus/gmavenplus/mojo/AbstractGenerateStubsMojo.java b/src/main/java/org/codehaus/gmavenplus/mojo/AbstractGenerateStubsMojo.java
@@ -46,6 +46,11 @@
  */
 public abstract class AbstractGenerateStubsMojo extends AbstractGroovyStubSourcesMojo {
 
+    /**
+     * Groovy 4.0.11 version.
+     */
+    protected static final Version GROOVY_4_0_11 = new Version(4, 0, 11);
+
     /**
      * Groovy 4.0.6 version.
      */
@@ -205,6 +210,7 @@ public abstract class AbstractGenerateStubsMojo extends AbstractGroovyStubSource
      * Using 18 requires Groovy &gt; 4.0.0-beta-1.
      * Using 19 requires Groovy &gt; 4.0.2.
      * Using 20 requires Groovy &gt; 4.0.6.
+     * Using 21 requires Groovy &gt; 4.0.11.
      *
      * @since 1.0-beta-3
      */
@@ -417,7 +423,11 @@ protected void resetStubModifiedDates(final Set<File> stubs) {
      * org.codehaus.groovy.classgen.asm.WriterController.
      */
     protected void verifyGroovyVersionSupportsTargetBytecode() {
-        if ("20".equals(targetBytecode)) {
+        if ("21".equals(targetBytecode)) {
+            if (groovyOlderThan(GROOVY_4_0_11)) {
+                throw new IllegalArgumentException("Target bytecode " + targetBytecode + " requires Groovy " + GROOVY_4_0_11 + " or newer.");
+            }
+        } else if ("20".equals(targetBytecode)) {
             if (groovyOlderThan(GROOVY_4_0_6)) {
                 throw new IllegalArgumentException("Target bytecode " + targetBytecode + " requires Groovy " + GROOVY_4_0_6 + " or newer.");
             }

diff --git a/src/test/java/org/codehaus/gmavenplus/mojo/AbstractCompileMojoTest.java b/src/test/java/org/codehaus/gmavenplus/mojo/AbstractCompileMojoTest.java
@@ -405,6 +405,13 @@ public void testJava20WithSupportedGroovy() {
         testMojo.verifyGroovyVersionSupportsTargetBytecode();
     }
 
+    @Test
+    public void testJava21WithSupportedGroovy() {
+        testMojo = new TestMojo("4.0.11");
+        testMojo.targetBytecode = "21";
+        testMojo.verifyGroovyVersionSupportsTargetBytecode();
+    }
+
     @Test(expected = IllegalArgumentException.class)
     public void testUnrecognizedJava() {
         testMojo = new TestMojo("2.1.2");

diff --git a/src/test/java/org/codehaus/gmavenplus/mojo/AbstractGenerateStubsMojoTest.java b/src/test/java/org/codehaus/gmavenplus/mojo/AbstractGenerateStubsMojoTest.java
@@ -432,6 +432,13 @@ public void testJava20WithSupportedGroovy() {
         testMojo.verifyGroovyVersionSupportsTargetBytecode();
     }
 
+    @Test
+    public void testJava21WithSupportedGroovy() {
+        testMojo = new TestMojo("4.0.11");
+        testMojo.targetBytecode = "21";
+        testMojo.verifyGroovyVersionSupportsTargetBytecode();
+    }
+
     @Test(expected = IllegalArgumentException.class)
     public void testUnrecognizedJava() {
         testMojo = new TestMojo("2.1.2");