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 2008/01/22 23:01:58 UTC

svn commit: r614342 - in /maven/components/trunk: maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java

Author: jdcasey
Date: Tue Jan 22 14:01:57 2008
New Revision: 614342

URL: http://svn.apache.org/viewvc?rev=614342&view=rev
Log:
Fixing error-reporting pointcut that I broke.

Modified:
    maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj
    maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java

Modified: maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj?rev=614342&r1=614341&r2=614342&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj (original)
+++ maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj Tue Jan 22 14:01:57 2008
@@ -18,6 +18,7 @@
 import org.apache.maven.lifecycle.model.MojoBinding;
 import org.apache.maven.lifecycle.statemgmt.StateManagementUtils;
 import org.apache.maven.lifecycle.DefaultLifecycleExecutor;
+import org.apache.maven.lifecycle.LifecycleExecutor;
 import org.apache.maven.lifecycle.LifecycleException;
 import org.apache.maven.lifecycle.LifecycleExecutionException;
 import org.apache.maven.project.MavenProject;
@@ -55,7 +56,9 @@
         && args( project, .. );
 
     after( MojoBinding binding, MavenProject project ) throwing ( PluginLoaderException cause ):
-        call( * PluginLoader+.loadPlugin( MojoBinding, MavenProject, .. ) )
+        ( cflow( le_executeGoalAndHandleFailures( MojoBinding ) )
+          || cflow( execution( * LifecycleExecutor+.isTaskValid( .. ) ) ) )
+        && call( * PluginLoader+.loadPlugin( MojoBinding, MavenProject, .. ) )
         && args( binding, project, .. )
     {
         getReporter().reportErrorLoadingPlugin( binding, project, cause );

Modified: maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java?rev=614342&r1=614341&r2=614342&view=diff
==============================================================================
--- maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java (original)
+++ maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java Tue Jan 22 14:01:57 2008
@@ -461,7 +461,7 @@
 
         reporter.reportErrorLoadingPlugin( null, null, null );
         reporterCtl.setMatcher( MockControl.ALWAYS_MATCHER );
-        reporterCtl.setVoidCallable();
+        reporterCtl.setVoidCallable( MockControl.ONE_OR_MORE );
 
         reporterCtl.replay();