Merge pull request #96 from groovy/fix-log-order

Fixed NPE when calling mojo without Groovy dependency

Author: keeganwitt

src/main/java/org/codehaus/gmavenplus/mojo/AbstractCompileMojo.java

@@ -165,14 +165,14 @@ protected synchronized void doCompile(final Set<File> sources, final List classp
             throws ClassNotFoundException, InstantiationException, IllegalAccessException, InvocationTargetException, MalformedURLException {
         classWrangler = new ClassWrangler(classpath, getLog());
 
-        logPluginClasspath();
-        classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
-
         if (sources == null || sources.isEmpty()) {
             getLog().info("No sources specified for compilation.  Skipping.");
             return;
         }
 
+        logPluginClasspath();
+        classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
+
         if (groovyVersionSupportsAction()) {
             verifyGroovyVersionSupportsTargetBytecode();
         } else {

src/main/java/org/codehaus/gmavenplus/mojo/AbstractGenerateStubsMojo.java

@@ -140,14 +140,14 @@ public abstract class AbstractGenerateStubsMojo extends AbstractGroovyStubSource
     protected synchronized void doStubGeneration(final Set<File> stubSources, final List classpath, final File outputDirectory) throws ClassNotFoundException, InvocationTargetException, IllegalAccessException, InstantiationException, MalformedURLException {
         classWrangler = new ClassWrangler(classpath, getLog());
 
-        logPluginClasspath();
-        classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
-
         if (stubSources == null || stubSources.isEmpty()) {
             getLog().info("No sources specified for stub generation.  Skipping.");
             return;
         }
 
+        logPluginClasspath();
+        classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
+
         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;

src/main/java/org/codehaus/gmavenplus/mojo/AbstractGroovyDocMojo.java

@@ -191,13 +191,16 @@ protected synchronized void doGroovyDocGeneration(final FileSet[] sourceDirector
             getLog().info("Skipping generation of GroovyDoc because ${maven.groovydoc.skip} was set to true.");
             return;
         }
+
         if (sourceDirectories == null || sourceDirectories.length == 0) {
             getLog().info("No source directories specified for GroovyDoc generation.  Skipping.");
             return;
         }
+
+        classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
+        logPluginClasspath();
+
         if (groovyVersionSupportsAction()) {
-            classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
-            logPluginClasspath();
             try {
                 getLog().debug("Project compile classpath:\n" + project.getCompileClasspathElements());
             } catch (DependencyResolutionRequiredException e) {

src/main/java/org/codehaus/gmavenplus/mojo/ConsoleMojo.java

@@ -61,6 +61,7 @@ public void execute() throws MojoExecutionException, MojoFailureException {
 
         logPluginClasspath();
         classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
+
         try {
             getLog().debug("Project test classpath:\n" + project.getTestClasspathElements());
         } catch (DependencyResolutionRequiredException e) {

src/main/java/org/codehaus/gmavenplus/mojo/ExecuteMojo.java

@@ -95,20 +95,21 @@ public void execute() throws MojoExecutionException, MojoFailureException {
     protected synchronized void doExecute() throws MojoExecutionException, MojoFailureException {
         classWrangler = new ClassWrangler(Thread.currentThread().getContextClassLoader(), getLog());
 
+        if (scripts == null || scripts.length == 0) {
+            getLog().info("No scripts specified for execution.  Skipping.");
+            return;
+        }
+
         logPluginClasspath();
         classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
+
         try {
             getLog().debug("Project test classpath:\n" + project.getTestClasspathElements());
         } catch (DependencyResolutionRequiredException e) {
             getLog().warn("Unable to log project test classpath", e);
         }
 
         if (groovyVersionSupportsAction()) {
-            if (scripts == null || scripts.length == 0) {
-                getLog().info("No scripts specified for execution.  Skipping.");
-                return;
-            }
-
             final SecurityManager sm = System.getSecurityManager();
             try {
                 if (!allowSystemExits) {

src/main/java/org/codehaus/gmavenplus/mojo/ShellMojo.java

@@ -73,6 +73,7 @@ public void execute() throws MojoExecutionException, MojoFailureException {
 
         logPluginClasspath();
         classWrangler.logGroovyVersion(mojoExecution.getMojoDescriptor().getGoal());
+
         try {
             getLog().debug("Project test classpath:\n" + project.getTestClasspathElements());
         } catch (DependencyResolutionRequiredException e) {