You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by be...@apache.org on 2011/02/11 21:04:37 UTC
svn commit: r1069935 -
/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
Author: bentmann
Date: Fri Feb 11 20:04:37 2011
New Revision: 1069935
URL: http://svn.apache.org/viewvc?rev=1069935&view=rev
Log:
o Allowed event spy to monitor project dependency graph
Modified:
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java?rev=1069935&r1=1069934&r2=1069935&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java Fri Feb 11 20:04:37 2011
@@ -19,6 +19,7 @@ import org.apache.maven.RepositoryUtils;
import org.apache.maven.artifact.Artifact;
import org.apache.maven.artifact.ArtifactUtils;
import org.apache.maven.artifact.factory.ArtifactFactory;
+import org.apache.maven.eventspy.internal.EventSpyDispatcher;
import org.apache.maven.execution.MavenSession;
import org.apache.maven.lifecycle.LifecycleExecutionException;
import org.apache.maven.project.DefaultDependencyResolutionRequest;
@@ -61,6 +62,9 @@ public class LifecycleDependencyResolver
@Requirement
private ArtifactFactory artifactFactory;
+ @Requirement
+ private EventSpyDispatcher eventSpyDispatcher;
+
@SuppressWarnings({"UnusedDeclaration"})
public LifecycleDependencyResolver()
{
@@ -162,6 +166,8 @@ public class LifecycleDependencyResolver
new DefaultDependencyResolutionRequest( project, session.getRepositorySession() );
request.setResolutionFilter( resolutionFilter );
+ eventSpyDispatcher.onEvent( request );
+
result = dependenciesResolver.resolve( request );
}
catch ( DependencyResolutionException e )
@@ -191,6 +197,8 @@ public class LifecycleDependencyResolver
}
}
+ eventSpyDispatcher.onEvent( result );
+
Set<Artifact> artifacts = new LinkedHashSet<Artifact>();
if ( result.getDependencyGraph() != null && !result.getDependencyGraph().getChildren().isEmpty() )
{
Re: svn commit: r1069935 - /maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
Posted by Olivier Lamy <ol...@apache.org>.
Hello,
I wonder if we could have some documentation in the javadoc of
EventSpy for the method :
void onEvent( Object event )
throws Exception;
Because without navigating in the code it's a little bit mysterious to
know what type of events are "monitored" here.
Thanks
--
Olivier Lamy
http://twitter.com/olamy
http://www.linkedin.com/in/olamy
2011/2/11 <be...@apache.org>:
> Author: bentmann
> Date: Fri Feb 11 20:04:37 2011
> New Revision: 1069935
>
> URL: http://svn.apache.org/viewvc?rev=1069935&view=rev
> Log:
> o Allowed event spy to monitor project dependency graph
>
> Modified:
> maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
>
> Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java
> URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java?rev=1069935&r1=1069934&r2=1069935&view=diff
> ==============================================================================
> --- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java (original)
> +++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/internal/LifecycleDependencyResolver.java Fri Feb 11 20:04:37 2011
> @@ -19,6 +19,7 @@ import org.apache.maven.RepositoryUtils;
> import org.apache.maven.artifact.Artifact;
> import org.apache.maven.artifact.ArtifactUtils;
> import org.apache.maven.artifact.factory.ArtifactFactory;
> +import org.apache.maven.eventspy.internal.EventSpyDispatcher;
> import org.apache.maven.execution.MavenSession;
> import org.apache.maven.lifecycle.LifecycleExecutionException;
> import org.apache.maven.project.DefaultDependencyResolutionRequest;
> @@ -61,6 +62,9 @@ public class LifecycleDependencyResolver
> @Requirement
> private ArtifactFactory artifactFactory;
>
> + @Requirement
> + private EventSpyDispatcher eventSpyDispatcher;
> +
> @SuppressWarnings({"UnusedDeclaration"})
> public LifecycleDependencyResolver()
> {
> @@ -162,6 +166,8 @@ public class LifecycleDependencyResolver
> new DefaultDependencyResolutionRequest( project, session.getRepositorySession() );
> request.setResolutionFilter( resolutionFilter );
>
> + eventSpyDispatcher.onEvent( request );
> +
> result = dependenciesResolver.resolve( request );
> }
> catch ( DependencyResolutionException e )
> @@ -191,6 +197,8 @@ public class LifecycleDependencyResolver
> }
> }
>
> + eventSpyDispatcher.onEvent( result );
> +
> Set<Artifact> artifacts = new LinkedHashSet<Artifact>();
> if ( result.getDependencyGraph() != null && !result.getDependencyGraph().getChildren().isEmpty() )
> {
>
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org