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

[ke...@apache.org]

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: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org