You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jd...@apache.org on 2005/10/12 07:05:42 UTC

svn commit: r314785 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/MojoExecutionExceptionDiagnoser.java

Author: jdcasey
Date: Tue Oct 11 22:05:39 2005
New Revision: 314785

URL: http://svn.apache.org/viewcvs?rev=314785&view=rev
Log:
Cleaning up diagnosis output for MojoExecutionException's.

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/MojoExecutionExceptionDiagnoser.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/MojoExecutionExceptionDiagnoser.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/MojoExecutionExceptionDiagnoser.java?rev=314785&r1=314784&r2=314785&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/MojoExecutionExceptionDiagnoser.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/MojoExecutionExceptionDiagnoser.java Tue Oct 11 22:05:39 2005
@@ -17,15 +17,34 @@
         
         StringBuffer message = new StringBuffer();
         
-        message.append( "Error executing mojo: " ).append( mee.getSource() ).append( "\n\n" );
-        message.append( mee.getLongMessage() ).append( "\n\n" );
+        message.append( "Error executing mojo" );
+        
+        Object source = mee.getSource();
+        if ( source != null )
+        {
+            message.append( ": " ).append( mee.getSource() ).append( "\n" );
+        }
+        else
+        {
+            message.append( ".\n" );
+        }
+        
+        message.append( "\nMessage: " ).append( mee.getMessage() );
+        
+        String longMessage = mee.getLongMessage();
+        if ( longMessage != null )
+        {
+            message.append( "\n\n" ).append( longMessage );
+        }
         
         Throwable root = DiagnosisUtils.getRootCause( mee );
         
         if ( root != null && root != mee )
         {
-            message.append( "Root Cause: " ).append( root.getMessage() ).append( "\n\n" );
+            message.append( "\n\nRoot Cause: " ).append( root.getMessage() );
         }
+        
+        message.append( "\n\n" );
         
         return message.toString();
     }