Author: jvanzyl
Date: Wed Feb 25 18:28:39 2009
New Revision: 747888

URL: http://svn.apache.org/viewvc?rev=747888&view=rev
Log: (empty)

Modified:
    
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
    
maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestPopulator.java
    
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
    
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java
    
maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/LegacyMavenRepositorySystem.java
    
maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/MavenRepositorySystem.java

Modified: 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java?rev=747888&r1=747887&r2=747888&view=diff
==============================================================================
--- 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
 (original)
+++ 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
 Wed Feb 25 18:28:39 2009
@@ -138,7 +138,7 @@
     protected PluginVersionManager pluginVersionManager;
 
     @Requirement
-    protected MavenRepositorySystem repositoryTools;
+    protected MavenRepositorySystem repositorySystem;
 
     @Requirement
     protected RuntimeInformation runtimeInformation;
@@ -393,7 +393,7 @@
         
         try
         {
-            projectPluginDependencies = repositoryTools.createArtifacts(
+            projectPluginDependencies = repositorySystem.createArtifacts(
                                                                              
plugin.getDependencies(),
                                                                              
null,
                                                                              
coreArtifactFilterManager.getCoreArtifactFilter(),
@@ -409,7 +409,7 @@
 
         try
         {
-            resolutionGroup = repositoryTools.retrieve( pluginArtifact, 
localRepository, project.getRemoteArtifactRepositories() );
+            resolutionGroup = repositorySystem.retrieve( pluginArtifact, 
localRepository, project.getRemoteArtifactRepositories() );
         }
         catch ( ArtifactMetadataRetrievalException e )
         {
@@ -458,9 +458,9 @@
             .setRemoteRepostories( repositories.isEmpty() ? 
Collections.EMPTY_LIST : new ArrayList( repositories ) )
             .setManagedVersionMap( pluginManagedDependencies )
             .setFilter( filter )                
-            .setMetadataSource( repositoryTools );        
+            .setMetadataSource( repositorySystem );        
         
-        ArtifactResolutionResult result = repositoryTools.resolve( request );
+        ArtifactResolutionResult result = repositorySystem.resolve( request );
 
         if ( result.hasErrorArtifactExceptions() )
         {
@@ -548,13 +548,13 @@
                 MavenProject p = (MavenProject) i.next();
 
                 resolveTransitiveDependencies( session,
-                                               repositoryTools,
+                                               repositorySystem,
                                                
mojoDescriptor.isDependencyResolutionRequired(),
                                                p,
                                                mojoDescriptor.isAggregator() );
             }
 
-            downloadDependencies( project, session, repositoryTools );
+            downloadDependencies( project, session, repositorySystem );
         }
 
         String goalName = mojoDescriptor.getFullGoalName();

Modified: 
maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestPopulator.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestPopulator.java?rev=747888&r1=747887&r2=747888&view=diff
==============================================================================
--- 
maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestPopulator.java
 (original)
+++ 
maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestPopulator.java
 Wed Feb 25 18:28:39 2009
@@ -208,8 +208,8 @@
 
                 profileManager.addProfile( profile );
 
-                // We need to convert profile repositories to artifact 
repositories
-
+                // We need to convert profile repositories to artifact 
repositories                
+                
                 for ( Iterator j = profile.getRepositories().iterator(); 
j.hasNext(); )
                 {
                     Repository r = (Repository) j.next();

Modified: 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java?rev=747888&r1=747887&r2=747888&view=diff
==============================================================================
--- 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
 (original)
+++ 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
 Wed Feb 25 18:28:39 2009
@@ -31,6 +31,8 @@
 import org.apache.maven.artifact.ArtifactUtils;
 import org.apache.maven.artifact.InvalidRepositoryException;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.resolver.ArtifactNotFoundException;
+import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.artifact.resolver.ArtifactResolutionRequest;
 import org.apache.maven.artifact.resolver.ArtifactResolutionResult;
 import org.apache.maven.model.Build;
@@ -194,7 +196,22 @@
         }
         
         File f = (artifact.getFile() != null) ? artifact.getFile() : new File( 
localRepository.getBasedir(), localRepository.pathOf( artifact ) );
-        repositorySystem.findModelFromRepository( artifact, 
artifactRepositories, localRepository );
+        try
+        {
+            repositorySystem.findModelFromRepository( artifact, 
artifactRepositories, localRepository );
+        }
+        catch ( ArtifactResolutionException e )
+        {
+            throw new ProjectBuildingException( project.getId(), "Error 
resolving project artifact.", e );
+        }
+        catch ( ArtifactNotFoundException e )
+        {
+            throw new ProjectBuildingException( project.getId(), "Error 
finding project artifact.", e );
+        }
+        catch ( InvalidRepositoryException e )
+        {
+            throw new ProjectBuildingException( project.getId(), "Error with 
repository specified in project.", e );
+        }
 
         ProjectBuilderConfiguration config = new 
DefaultProjectBuilderConfiguration().setLocalRepository( localRepository );
 

Modified: 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java?rev=747888&r1=747887&r2=747888&view=diff
==============================================================================
--- 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java
 (original)
+++ 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java
 Wed Feb 25 18:28:39 2009
@@ -19,6 +19,14 @@
  * under the License.
  */
 
+import java.io.File;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.Iterator;
+import java.util.LinkedHashSet;
+import java.util.List;
+import java.util.Set;
+
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.factory.ArtifactFactory;
 import org.apache.maven.artifact.metadata.ArtifactMetadataRetrievalException;
@@ -41,27 +49,19 @@
 import org.apache.maven.model.DistributionManagement;
 import org.apache.maven.model.Exclusion;
 import org.apache.maven.model.Relocation;
-import org.apache.maven.profiles.activation.ProfileActivator;
 import org.apache.maven.project.DefaultProjectBuilderConfiguration;
 import org.apache.maven.project.InvalidProjectModelException;
 import org.apache.maven.project.MavenProject;
 import org.apache.maven.project.MavenProjectBuilder;
 import org.apache.maven.project.ProjectBuildingException;
 import org.apache.maven.project.validation.ModelValidationResult;
-import org.codehaus.plexus.PlexusConstants;
 import org.codehaus.plexus.PlexusContainer;
 import org.codehaus.plexus.component.annotations.Component;
 import org.codehaus.plexus.component.annotations.Requirement;
 import 
org.codehaus.plexus.component.repository.exception.ComponentLookupException;
-import org.codehaus.plexus.context.Context;
-import org.codehaus.plexus.context.ContextException;
 import org.codehaus.plexus.logging.AbstractLogEnabled;
-import org.codehaus.plexus.personality.plexus.lifecycle.phase.Contextualizable;
 import org.codehaus.plexus.util.StringUtils;
 
-import java.io.File;
-import java.util.*;
-
 /**
  * @author Jason van Zyl
  * @author <a href="mailto:br...@apache.org";>Brett Porter</a>
@@ -70,7 +70,7 @@
 @Component(role = ArtifactMetadataSource.class )
 public class MavenMetadataSource
     extends AbstractLogEnabled
-    implements ArtifactMetadataSource, Contextualizable
+    implements ArtifactMetadataSource
 {
     public static final String ROLE_HINT = "default";
 
@@ -594,12 +594,6 @@
         return versions;
     }
 
-    public void contextualize( Context context )
-        throws ContextException
-    {
-        container = (PlexusContainer) context.get( PlexusConstants.PLEXUS_KEY 
);
-    }
-
     private static final class ProjectRelocation
     {
         private MavenProject project;

Modified: 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/LegacyMavenRepositorySystem.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/LegacyMavenRepositorySystem.java?rev=747888&r1=747887&r2=747888&view=diff
==============================================================================
--- 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/LegacyMavenRepositorySystem.java
 (original)
+++ 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/LegacyMavenRepositorySystem.java
 Wed Feb 25 18:28:39 2009
@@ -53,7 +53,6 @@
 import org.apache.maven.model.Repository;
 import org.apache.maven.model.RepositoryPolicy;
 import org.apache.maven.project.MavenProject;
-import org.apache.maven.project.ProjectBuildingException;
 import org.apache.maven.project.artifact.InvalidDependencyVersionException;
 import org.apache.maven.project.artifact.MavenMetadataSource;
 import org.apache.maven.wagon.authentication.AuthenticationInfo;
@@ -128,7 +127,7 @@
         {
             return null;
         }
-        
+
         return artifactFactory.createDependencyArtifact( groupId, artifactId, 
versionRange, type, classifier, scope );
     }
 
@@ -143,7 +142,7 @@
         {
             return null;
         }
-        
+
         return artifactFactory.createDependencyArtifact( groupId, artifactId, 
versionRange, type, classifier, scope, inheritedScope );
     }
 
@@ -160,8 +159,8 @@
         }
 
         return artifactFactory.createExtensionArtifact( groupId, artifactId, 
versionRange );
-    }    
-    
+    }
+
     public Artifact createParentArtifact( String groupId, String artifactId, 
String version )
     {
         return artifactFactory.createParentArtifact( groupId, artifactId, 
version );
@@ -178,7 +177,7 @@
         {
             return null;
         }
-        
+
         return artifactFactory.createPluginArtifact( groupId, artifactId, 
versionRange );
     }
 
@@ -348,7 +347,7 @@
     // Taken from RepositoryHelper
 
     public void findModelFromRepository( Artifact artifact, List 
remoteArtifactRepositories, ArtifactRepository localRepository )
-        throws ProjectBuildingException
+        throws InvalidRepositoryException, ArtifactResolutionException, 
ArtifactNotFoundException    
     {
 
         if ( cache.containsKey( artifact.getId() ) )
@@ -374,26 +373,15 @@
             projectArtifact = artifactFactory.createProjectArtifact( 
artifact.getGroupId(), artifact.getArtifactId(), artifact.getVersion(), 
artifact.getScope() );
         }
 
-        try
-        {
-            artifactResolver.resolve( projectArtifact, 
remoteArtifactRepositories, localRepository );
+        artifactResolver.resolve( projectArtifact, remoteArtifactRepositories, 
localRepository );
 
-            File file = projectArtifact.getFile();
-            artifact.setFile( file );
-            cache.put( artifact.getId(), artifact );
-        }
-        catch ( ArtifactResolutionException e )
-        {
-            throw new ProjectBuildingException( projectId, "Error getting POM 
for '" + projectId + "' from the repository: " + e.getMessage(), e );
-        }
-        catch ( ArtifactNotFoundException e )
-        {
-            throw new ProjectBuildingException( projectId, "POM '" + projectId 
+ "' not found in repository: " + e.getMessage(), e );
-        }
+        File file = projectArtifact.getFile();
+        artifact.setFile( file );
+        cache.put( artifact.getId(), artifact );
     }
 
     private List normalizeToArtifactRepositories( List 
remoteArtifactRepositories, String projectId )
-        throws ProjectBuildingException
+        throws InvalidRepositoryException
     {
         List normalized = new ArrayList( remoteArtifactRepositories.size() );
 
@@ -409,21 +397,14 @@
             else if ( item instanceof Repository )
             {
                 Repository repo = (Repository) item;
-                try
-                {
-                    item = buildArtifactRepository( repo );
+                item = buildArtifactRepository( repo );
 
-                    normalized.add( item );
-                    normalizationNeeded = true;
-                }
-                catch ( InvalidRepositoryException e )
-                {
-                    throw new ProjectBuildingException( projectId, "Error 
building artifact repository for id: " + repo.getId(), e );
-                }
+                normalized.add( item );
+                normalizationNeeded = true;
             }
             else
             {
-                throw new ProjectBuildingException( projectId, "Error building 
artifact repository from non-repository information item: " + item );
+                throw new InvalidRepositoryException( projectId, "Error 
building artifact repository from non-repository information item: " + item );
             }
         }
 
@@ -471,8 +452,8 @@
     public ArtifactResolutionResult resolve( ArtifactResolutionRequest request 
)
     {
         return artifactResolver.resolve( request );
-    }    
-    
+    }
+
     // ------------------------------------------------------------------------
     // Extracted from DefaultWagonManager
     // ------------------------------------------------------------------------

Modified: 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/MavenRepositorySystem.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/MavenRepositorySystem.java?rev=747888&r1=747887&r2=747888&view=diff
==============================================================================
--- 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/MavenRepositorySystem.java
 (original)
+++ 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/repository/MavenRepositorySystem.java
 Wed Feb 25 18:28:39 2009
@@ -17,7 +17,6 @@
 
 import java.io.IOException;
 import java.util.List;
-import java.util.Map;
 import java.util.Set;
 
 import org.apache.maven.artifact.Artifact;
@@ -26,20 +25,15 @@
 import org.apache.maven.artifact.metadata.ArtifactMetadataSource;
 import org.apache.maven.artifact.metadata.ResolutionGroup;
 import org.apache.maven.artifact.repository.ArtifactRepository;
-import org.apache.maven.artifact.repository.ArtifactRepositoryPolicy;
 import org.apache.maven.artifact.resolver.ArtifactNotFoundException;
 import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.artifact.resolver.ArtifactResolutionRequest;
 import org.apache.maven.artifact.resolver.ArtifactResolutionResult;
 import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
 import org.apache.maven.artifact.versioning.ArtifactVersion;
-import org.apache.maven.artifact.versioning.VersionRange;
 import org.apache.maven.model.Dependency;
-import org.apache.maven.model.DeploymentRepository;
-import org.apache.maven.model.Model;
 import org.apache.maven.model.Repository;
 import org.apache.maven.project.MavenProject;
-import org.apache.maven.project.ProjectBuildingException;
 import org.apache.maven.project.artifact.InvalidDependencyVersionException;
 import org.apache.maven.wagon.events.TransferListener;
 
@@ -93,6 +87,8 @@
 
     // Artifact resolution
 
+    //MetadataResolutionResult resolveMetadata( MetadataResolutionRequest 
request );
+
     ArtifactResolutionResult resolve( ArtifactResolutionRequest request );
 
     // This can be reduced to the request/result
@@ -100,7 +96,7 @@
         throws ArtifactResolutionException, ArtifactNotFoundException;
 
     void findModelFromRepository( Artifact artifact, List 
remoteArtifactRepositories, ArtifactRepository localRepository )
-        throws ProjectBuildingException;
+        throws InvalidRepositoryException, ArtifactResolutionException, 
ArtifactNotFoundException;
 
     // Version retrieval or metadata operations
 


Reply via email to