Fix bytecode check condition (closes #276)

keeganwitt · keeganwitt · commit 4e44955cd167 · 2023-09-28T09:54:03.000-04:00
diff --git a/src/main/java/org/codehaus/gmavenplus/mojo/AbstractCompileMojo.java b/src/main/java/org/codehaus/gmavenplus/mojo/AbstractCompileMojo.java
@@ -337,13 +337,15 @@ protected synchronized void doCompile(final Set<File> sources, final List classp
         logPluginClasspath();
         classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
 
-        if (groovyVersionSupportsAction() && !skipBytecodeCheck) {
-            verifyGroovyVersionSupportsTargetBytecode();
-        } else {
+        if (!groovyVersionSupportsAction()) {
             getLog().error("Your Groovy version (" + classWrangler.getGroovyVersionString() + ") doesn't support compilation. The minimum version of Groovy required is " + minGroovyVersion + ". Skipping compiling.");
             return;
         }
 
+        if (!skipBytecodeCheck) {
+            verifyGroovyVersionSupportsTargetBytecode();
+        }
+
         // get classes we need with reflection
         Class<?> compilerConfigurationClass = classWrangler.getClass("org.codehaus.groovy.control.CompilerConfiguration");
         Class<?> compilationUnitClass = classWrangler.getClass("org.codehaus.groovy.control.CompilationUnit");
diff --git a/src/main/java/org/codehaus/gmavenplus/mojo/AbstractGenerateStubsMojo.java b/src/main/java/org/codehaus/gmavenplus/mojo/AbstractGenerateStubsMojo.java
@@ -295,13 +295,15 @@ protected synchronized void doStubGeneration(final Set<File> stubSources, final
         logPluginClasspath();
         classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
 
-        if (groovyVersionSupportsAction() && !skipBytecodeCheck) {
-            verifyGroovyVersionSupportsTargetBytecode();
-        } else {
+        if (!groovyVersionSupportsAction()) {
             getLog().error("Your Groovy version (" + classWrangler.getGroovyVersionString() + ") doesn't support stub generation. The minimum version of Groovy required is " + minGroovyVersion + ". Skipping stub generation.");
             return;
         }
 
+        if (!skipBytecodeCheck) {
+            verifyGroovyVersionSupportsTargetBytecode();
+        }
+
         // get classes we need with reflection
         Class<?> compilerConfigurationClass = classWrangler.getClass("org.codehaus.groovy.control.CompilerConfiguration");
         Class<?> javaStubCompilationUnitClass = classWrangler.getClass("org.codehaus.groovy.tools.javac.JavaStubCompilationUnit");
