Author: bentmann
Date: Tue Oct 20 09:50:35 2009
New Revision: 827020

URL: http://svn.apache.org/viewvc?rev=827020&view=rev
Log:
o Specialized exception

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

Modified: 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java
URL: 
http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java?rev=827020&r1=827019&r2=827020&view=diff
==============================================================================
--- 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java
 (original)
+++ 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultModelBuildingListener.java
 Tue Oct 20 09:50:35 2009
@@ -24,10 +24,10 @@
 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;
 import org.apache.maven.model.building.ModelBuildingEvent;
+import org.apache.maven.plugin.PluginResolutionException;
 import org.apache.maven.plugin.version.PluginVersionResolutionException;
 
 /**
@@ -116,7 +116,7 @@
                 project.setClassRealm( record.realm );
                 project.setExtensionArtifactFilter( 
record.extensionArtifactFilter );
             }
-            catch ( ArtifactResolutionException e )
+            catch ( PluginResolutionException e )
             {
                 event.getProblems().addError( "Unresolveable build extensions: 
" + e.getMessage(), e );
             }

Modified: 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java
URL: 
http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java?rev=827020&r1=827019&r2=827020&view=diff
==============================================================================
--- 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java
 (original)
+++ 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultProjectBuildingHelper.java
 Tue Oct 20 09:50:35 2009
@@ -50,6 +50,7 @@
 import org.apache.maven.model.Repository;
 import org.apache.maven.plugin.ExtensionRealmCache;
 import org.apache.maven.plugin.PluginArtifactsCache;
+import org.apache.maven.plugin.PluginResolutionException;
 import org.apache.maven.plugin.version.DefaultPluginVersionRequest;
 import org.apache.maven.plugin.version.PluginVersionRequest;
 import org.apache.maven.plugin.version.PluginVersionResolutionException;
@@ -135,7 +136,7 @@
 
     public synchronized ProjectRealmCache.CacheRecord createProjectRealm( 
MavenProject project, Model model,
                                                                           
RepositoryRequest repositoryRequest )
-        throws ArtifactResolutionException, PluginVersionResolutionException
+        throws PluginResolutionException, PluginVersionResolutionException
     {
         ClassRealm projectRealm = null;
 
@@ -381,7 +382,7 @@
     }
 
     private List<Artifact> resolveExtensionArtifacts( Plugin extensionPlugin, 
RepositoryRequest repositoryRequest )
-        throws ArtifactResolutionException
+        throws PluginResolutionException
     {
         Artifact extensionArtifact = repositorySystem.createPluginArtifact( 
extensionPlugin );
 
@@ -405,7 +406,14 @@
 
         ArtifactResolutionResult result = repositorySystem.resolve( request );
 
-        resolutionErrorHandler.throwErrors( request, result );
+        try
+        {
+            resolutionErrorHandler.throwErrors( request, result );
+        }
+        catch ( ArtifactResolutionException e )
+        {
+            throw new PluginResolutionException( extensionPlugin, e );
+        }
 
         List<Artifact> extensionArtifacts = new ArrayList<Artifact>( 
result.getArtifacts() );
 

Modified: 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java
URL: 
http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java?rev=827020&r1=827019&r2=827020&view=diff
==============================================================================
--- 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java
 (original)
+++ 
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/ProjectBuildingHelper.java
 Tue Oct 20 09:50:35 2009
@@ -24,9 +24,9 @@
 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;
+import org.apache.maven.plugin.PluginResolutionException;
 import org.apache.maven.plugin.version.PluginVersionResolutionException;
 
 /**
@@ -62,10 +62,10 @@
      * @param model The model to create the project realm for, must not be 
{...@code null}
      * @param repositoryRequest The repository request to use for artifact 
resolution, must not be {...@code null}.
      * @return The record with the project realm and extension artifact 
filter, never {...@code null}.
-     * @throws ArtifactResolutionException If any build extension could not be 
resolved.
+     * @throws PluginResolutionException If any build extension could not be 
resolved.
      */
     ProjectRealmCache.CacheRecord createProjectRealm( MavenProject project, 
Model model,
                                                       RepositoryRequest 
repositoryRequest )
-        throws ArtifactResolutionException, PluginVersionResolutionException;
+        throws PluginResolutionException, PluginVersionResolutionException;
 
 }

Modified: 
maven/maven-3/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java
URL: 
http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java?rev=827020&r1=827019&r2=827020&view=diff
==============================================================================
--- 
maven/maven-3/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java
 (original)
+++ 
maven/maven-3/trunk/maven-core/src/test/java/org/apache/maven/project/EmptyProjectBuildingHelper.java
 Tue Oct 20 09:50:35 2009
@@ -22,10 +22,8 @@
 import java.util.ArrayList;
 import java.util.List;
 
-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;
 import org.codehaus.plexus.component.annotations.Component;
@@ -43,7 +41,6 @@
     public List<ArtifactRepository> createArtifactRepositories( 
List<Repository> pomRepositories,
                                                                 
List<ArtifactRepository> externalRepositories,
                                                                 
ProjectBuildingRequest request )
-        throws InvalidRepositoryException
     {
         if ( externalRepositories != null )
         {
@@ -57,7 +54,6 @@
 
     public ProjectRealmCache.CacheRecord createProjectRealm( MavenProject 
proejct, Model model,
                                                              RepositoryRequest 
repositoryRequest )
-        throws ArtifactResolutionException
     {
         return new ProjectRealmCache.CacheRecord( null, null );
     }


Reply via email to