svn commit: r516998 - /maven/maven-1/core/trunk/src/java/org/apache/maven/cli/App.java

[ltheussl]

Author: ltheussl
Date: Sun Mar 11 12:41:00 2007
New Revision: 516998

URL: http://svn.apache.org/viewvc?view=rev&rev=516998
Log:
PR: MAVEN-1816
Confusing console output on test failures.

Modified:
    maven/maven-1/core/trunk/src/java/org/apache/maven/cli/App.java

Modified: maven/maven-1/core/trunk/src/java/org/apache/maven/cli/App.java
URL: 
http://svn.apache.org/viewvc/maven/maven-1/core/trunk/src/java/org/apache/maven/cli/App.java?view=diff&rev=516998&r1=516997&r2=516998
==============================================================================
--- maven/maven-1/core/trunk/src/java/org/apache/maven/cli/App.java (original)
+++ maven/maven-1/core/trunk/src/java/org/apache/maven/cli/App.java Sun Mar 11 
12:41:00 2007
@@ -356,11 +356,14 @@
             failed = true;
             displayBuildFailed();
             MavenUtils.displayThrowable( e, displayStackTrace );
-            displayBugReportHelp();
             if ( e.getCause() instanceof JellyException )
+            {
                 returnCode = RC_JELLY_FAILED;
+            }
             else
+            {
                 returnCode = RC_GOAL_FAILED;
+            }
         }
         catch ( JellyException e )
         {
@@ -376,7 +379,9 @@
             displayBuildFailed();
             MavenUtils.displayThrowable( e, displayStackTrace );
             if ( MavenUtils.MAVEN_UNKNOWN_ERROR.equals( e.getMessage() ) )
+            {
                 displayBugReportHelp();
+            }
             returnCode = RC_OTHER_FAILURE;
 
         }
@@ -1267,4 +1272,5 @@
         LOGGER.error( MavenUtils.getMessage( "build.errors.stack" ) );
 
     }
+
 }