Merge pull request #211 from groovy/210

Throw exception instead of returning null when Groovy can't be found (closes #210)
groovy · Feb 15, 2022 · 8f7f74e · 8f7f74e
2 parents 87fd2ca + 6545e00
commit 8f7f74e
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/main/java/org/codehaus/gmavenplus/util/ClassWrangler.java b/src/main/java/org/codehaus/gmavenplus/util/ClassWrangler.java
@@ -141,8 +141,7 @@ public Version getGroovyVersion() {
         try {
             return Version.parseFromString(getGroovyVersionString());
         } catch (Exception e) {
-            log.error("Unable to determine Groovy version. Is Groovy declared as a dependency?");
-            return null;
+            throw new RuntimeException("Unable to determine Groovy version. Is Groovy declared as a dependency?");
         }
     }
 
@@ -279,8 +278,7 @@ protected String getGroovyJar() {
 
             return groovyJar;
         } catch (ClassNotFoundException e) {
-            log.error("Unable to determine Groovy version. Is Groovy declared as a dependency?");
-            return null;
+            throw new RuntimeException("Unable to determine Groovy version. Is Groovy declared as a dependency?");
         }
     }