You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by be...@apache.org on 2009/08/06 12:24:06 UTC

svn commit: r801591 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/exception/DefaultExceptionHandler.java

Author: bentmann
Date: Thu Aug  6 10:24:05 2009
New Revision: 801591

URL: http://svn.apache.org/viewvc?rev=801591&view=rev
Log:
o Fixed exception reporting

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/exception/DefaultExceptionHandler.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/exception/DefaultExceptionHandler.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/exception/DefaultExceptionHandler.java?rev=801591&r1=801590&r2=801591&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/exception/DefaultExceptionHandler.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/exception/DefaultExceptionHandler.java Thu Aug  6 10:24:05 2009
@@ -30,6 +30,7 @@
 import org.apache.maven.plugin.PluginNotFoundException;
 import org.apache.maven.plugin.PluginResolutionException;
 import org.codehaus.plexus.component.annotations.Component;
+import org.codehaus.plexus.util.StringUtils;
 
 /*
 
@@ -124,10 +125,20 @@
         else if ( exception instanceof MojoFailureException )
         {
             message = ((MojoFailureException)exception).getLongMessage();
+
+            if ( StringUtils.isEmpty( message ) )
+            {
+                message = exception.getMessage();
+            }
         }
         else if ( exception instanceof MojoExecutionException )
         {
             message = ((MojoExecutionException)exception).getLongMessage();
+
+            if ( StringUtils.isEmpty( message ) )
+            {
+                message = exception.getMessage();
+            }
         }
         
         else