Author: kenney
Date: Sun Sep 25 08:07:39 2005
New Revision: 291423

URL: http://svn.apache.org/viewcvs?rev=291423&view=rev
Log:
Since stacktraces are printed when a cause is present in MojoExecutionException,
log the error instead and throw an exception without a cause.

Maybe it's better to have 2 MojoExecutionExceptions: one that signals
a real error/failure in maven (or pom setup), and one that just
signals normal build errors (no stacktrace printed).

Modified:
    
maven/components/trunk/sandbox/maven-it-plugin/src/main/java/org/apache/maven/plugin/it/ForkMojo.java

Modified: 
maven/components/trunk/sandbox/maven-it-plugin/src/main/java/org/apache/maven/plugin/it/ForkMojo.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/sandbox/maven-it-plugin/src/main/java/org/apache/maven/plugin/it/ForkMojo.java?rev=291423&r1=291422&r2=291423&view=diff
==============================================================================
--- 
maven/components/trunk/sandbox/maven-it-plugin/src/main/java/org/apache/maven/plugin/it/ForkMojo.java
 (original)
+++ 
maven/components/trunk/sandbox/maven-it-plugin/src/main/java/org/apache/maven/plugin/it/ForkMojo.java
 Sun Sep 25 08:07:39 2005
@@ -118,8 +118,8 @@
 
         if ( response.isExecutionFailure() )
         {
-            throw new MojoExecutionException(
-                "Integration test failed" );
+            getLog().error( "Integration test failed", response.getException() 
);
+            throw new MojoExecutionException( "Integration test failed" );
         }
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to