You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by pr...@apache.org on 2006/12/14 20:28:50 UTC

svn commit: r487316 - /geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java

Author: prasad
Date: Thu Dec 14 11:28:49 2006
New Revision: 487316

URL: http://svn.apache.org/viewvc?view=rev&rev=487316
Log:
* if no build number specified, skip this goal.

Modified:
    geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java

Modified: geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java?view=diff&rev=487316&r1=487315&r2=487316
==============================================================================
--- geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java (original)
+++ geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java Thu Dec 14 11:28:49 2006
@@ -232,10 +232,15 @@
      */
     protected void doExecute() throws Exception {
 
+        if (buildNumber == null) {
+            log.warn("No build number specified; returning");
+            return;
+        }
+
         File currentSiteDirectory = new File(targetDirectory, "/site");
         if ( !currentSiteDirectory.exists() )
         {
-            log.warn("No site directory here");
+            log.warn("No site directory here; returning");
             return;
         }