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 2009/08/11 21:32:27 UTC

svn commit: r803256 - in /maven/components/trunk/maven-core/src: main/java/org/apache/maven/project/ test/java/org/apache/maven/project/

Author: bentmann
Date: Tue Aug 11 19:32:27 2009
New Revision: 803256

URL: http://svn.apache.org/viewvc?rev=803256&view=rev
Log:
o Fixed project building helper to use repo cache

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java
    maven/components/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java?rev=803256&r1=803255&r2=803256&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java Tue Aug 11 19:32:27 2009
@@ -22,6 +22,8 @@
 import java.util.List;
 
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.repository.DefaultRepositoryRequest;
+import org.apache.maven.artifact.repository.RepositoryRequest;
 import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.model.Model;
 import org.apache.maven.model.building.AbstractModelBuildingListener;
@@ -132,9 +134,13 @@
         {
             try
             {
-                projectRealm =
-                    projectBuildingHelper.createProjectRealm( model, projectBuildingRequest.getLocalRepository(),
-                                                              pluginRepositories );
+                RepositoryRequest repositoryRequest = new DefaultRepositoryRequest();
+                repositoryRequest.setCache( projectBuildingRequest.getRepositoryCache() );
+                repositoryRequest.setLocalRepository( projectBuildingRequest.getLocalRepository() );
+                repositoryRequest.setRemoteRepositories( pluginRepositories );
+                repositoryRequest.setOffline( projectBuildingRequest.isOffline() );
+
+                projectRealm = projectBuildingHelper.createProjectRealm( model, repositoryRequest );
             }
             catch ( ArtifactResolutionException e )
             {

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java?rev=803256&r1=803255&r2=803256&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java Tue Aug 11 19:32:27 2009
@@ -29,6 +29,7 @@
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.InvalidRepositoryException;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.repository.RepositoryRequest;
 import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.artifact.resolver.ArtifactResolutionRequest;
 import org.apache.maven.artifact.resolver.ArtifactResolutionResult;
@@ -101,8 +102,7 @@
         return artifactRepositories;
     }
 
-    public ClassRealm createProjectRealm( Model model, ArtifactRepository localRepository,
-                                          List<ArtifactRepository> remoteRepositories )
+    public ClassRealm createProjectRealm( Model model, RepositoryRequest repositoryRequest )
         throws ArtifactResolutionException
     {
         ClassRealm projectRealm = null;
@@ -137,7 +137,7 @@
                 repositorySystem.createArtifact( extension.getGroupId(), extension.getArtifactId(),
                                                  extension.getVersion(), "jar" );
 
-            populateRealm( projectRealm, artifact, null, localRepository, remoteRepositories );
+            populateRealm( projectRealm, artifact, null, repositoryRequest );
         }
 
         for ( Plugin plugin : extensionPlugins )
@@ -150,7 +150,7 @@
                 dependencies.add( repositorySystem.createDependencyArtifact( dependency ) );
             }
 
-            populateRealm( projectRealm, artifact, dependencies, localRepository, remoteRepositories );
+            populateRealm( projectRealm, artifact, dependencies, repositoryRequest );
         }
 
         try
@@ -167,15 +167,13 @@
     }
 
     private void populateRealm( ClassRealm realm, Artifact artifact, Set<Artifact> dependencies,
-                                ArtifactRepository localRepository, List<ArtifactRepository> remoteRepositories )
+                                RepositoryRequest repositoryRequest )
         throws ArtifactResolutionException
     {
-        ArtifactResolutionRequest request = new ArtifactResolutionRequest();
+        ArtifactResolutionRequest request = new ArtifactResolutionRequest( repositoryRequest );
         request.setArtifact( artifact );
         request.setArtifactDependencies( dependencies );
         request.setResolveTransitively( true );
-        request.setLocalRepository( localRepository );
-        request.setRemoteRepositories( remoteRepositories );
         // FIXME setTransferListener
 
         ArtifactResolutionResult result = repositorySystem.resolve( request );

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java?rev=803256&r1=803255&r2=803256&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java Tue Aug 11 19:32:27 2009
@@ -23,6 +23,7 @@
 
 import org.apache.maven.artifact.InvalidRepositoryException;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.repository.RepositoryRequest;
 import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.model.Model;
 import org.apache.maven.model.Repository;
@@ -55,13 +56,11 @@
      * Creates the project realm that hosts the build extensions of the specified model.
      * 
      * @param model The model to create the project realm for, must not be {@code null}
-     * @param localRepository The local repository to use for artifact resolution, must not be {@code null}.
-     * @param remoteRepositories The remote repositories to use for artifact resolution, must not be {@code null}.
+     * @param repositoryRequest The repository request to use for artifact resolution, must not be {@code null}.
      * @return The project realm or {@code null} if the project uses no extensions.
      * @throws ArtifactResolutionException If any build extension could not be resolved.
      */
-    ClassRealm createProjectRealm( Model model, ArtifactRepository localRepository,
-                                   List<ArtifactRepository> remoteRepositories )
+    ClassRealm createProjectRealm( Model model, RepositoryRequest repositoryRequest )
         throws ArtifactResolutionException;
 
 }

Modified: maven/components/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java?rev=803256&r1=803255&r2=803256&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java (original)
+++ maven/components/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java Tue Aug 11 19:32:27 2009
@@ -24,6 +24,7 @@
 
 import org.apache.maven.artifact.InvalidRepositoryException;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.repository.RepositoryRequest;
 import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.model.Model;
 import org.apache.maven.model.Repository;
@@ -54,8 +55,7 @@
         }
     }
 
-    public ClassRealm createProjectRealm( Model model, ArtifactRepository localRepository,
-                                       List<ArtifactRepository> remoteRepositories )
+    public ClassRealm createProjectRealm( Model model, RepositoryRequest repositoryRequest )
         throws ArtifactResolutionException
     {
         return null;