You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by br...@apache.org on 2005/05/11 04:13:19 UTC

svn commit: r169557 - /maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant /maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata /maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver /maven/components/trunk/maven-core/src/main/java/org/apache/maven /maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin /maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability /maven/components/trunk/maven-project/src/main/java/org/apache/maven/project /maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact

Author: brett
Date: Tue May 10 19:13:18 2005
New Revision: 169557

URL: http://svn.apache.org/viewcvs?rev=169557&view=rev
Log:
propogate more ArtifactResolutionExceptions directly

Modified:
    maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/Pom.java
    maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadataSource.java
    maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionException.java
    maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolver.java
    maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
    maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/TransitiveArtifactResolutionException.java
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/PluginNotFoundException.java
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/ArtifactResolverDiagnoser.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/MavenProjectBuilder.java
    maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java

Modified: maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/Pom.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/Pom.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/Pom.java (original)
+++ maven/components/trunk/maven-artifact-ant/src/main/java/org/apache/maven/artifact/ant/Pom.java Tue May 10 19:13:18 2005
@@ -17,6 +17,7 @@
  */
 
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.model.Build;
 import org.apache.maven.model.CiManagement;
 import org.apache.maven.model.DependencyManagement;
@@ -87,6 +88,10 @@
                 mavenProject = builder.build( file, localRepository );
             }
             catch ( ProjectBuildingException e )
+            {
+                throw new BuildException( "Unable to build project: " + file, e );
+            }
+            catch ( ArtifactResolutionException e )
             {
                 throw new BuildException( "Unable to build project: " + file, e );
             }

Modified: maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadataSource.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadataSource.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadataSource.java (original)
+++ maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/metadata/ArtifactMetadataSource.java Tue May 10 19:13:18 2005
@@ -17,6 +17,7 @@
  */
 
 import org.apache.maven.artifact.Artifact;
+import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 
 import java.util.List;
@@ -29,5 +30,5 @@
 public interface ArtifactMetadataSource
 {
     Set retrieve( Artifact artifact, ArtifactRepository localRepository, List remoteRepositories )
-        throws ArtifactMetadataRetrievalException;
+        throws ArtifactMetadataRetrievalException, ArtifactResolutionException;
 }

Modified: maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionException.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionException.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionException.java (original)
+++ maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionException.java Tue May 10 19:13:18 2005
@@ -1,6 +1,10 @@
 package org.apache.maven.artifact.resolver;
 
 import org.apache.maven.artifact.Artifact;
+import org.apache.maven.artifact.repository.ArtifactRepository;
+
+import java.util.Iterator;
+import java.util.List;
 
 /*
  * Copyright 2001-2005 The Apache Software Foundation.
@@ -33,19 +37,55 @@
 
     private String type;
 
-    public ArtifactResolutionException( String message, String groupId, String artifactId, String version, String type, Throwable t )
+    private List remoteRepositories;
+
+    public ArtifactResolutionException( String message, String groupId, String artifactId, String version, String type,
+                                        List remoteRepositories, Throwable t )
     {
-        super( "Unable to resolve artifact " + groupId + ":" + artifactId + ":" + version + ":" + type + "\n" + message, t );
+        super( constructMessage( message, groupId, artifactId, version, type, remoteRepositories ), t );
 
         this.groupId = groupId;
         this.artifactId = artifactId;
         this.type = type;
         this.version = version;
+        this.remoteRepositories = remoteRepositories;
     }
 
-    public ArtifactResolutionException( String message, Artifact artifact, Throwable t )
+    private static final String LS = System.getProperty( "line.separator" );
+
+    private static String constructMessage( String message, String groupId, String artifactId, String version,
+                                            String type, List remoteRepositories )
     {
-        this( message, artifact.getGroupId(), artifact.getArtifactId(), artifact.getVersion(), artifact.getType(), t );
+        StringBuffer sb = new StringBuffer();
+
+        sb.append( message );
+        sb.append( LS );
+        sb.append( LS );
+        sb.append( groupId + ":" + artifactId + ":" + version + ":" + type );
+        sb.append( LS );
+        sb.append( LS );
+        sb.append( "from the specified remote repositories:" );
+        sb.append( LS );
+        sb.append( LS );
+
+        for ( Iterator i = remoteRepositories.iterator(); i.hasNext(); )
+        {
+            ArtifactRepository remoteRepository = (ArtifactRepository) i.next();
+
+            sb.append( remoteRepository.getUrl() );
+            if ( i.hasNext() )
+            {
+                sb.append( ", " );
+            }
+        }
+
+        return sb.toString();
+    }
+
+    public ArtifactResolutionException( String message, Artifact artifact, List remoteRepositories, Throwable t )
+    {
+        this( message, artifact.getGroupId(), artifact.getArtifactId(), artifact.getVersion(), artifact.getType(),
+              remoteRepositories, t );
     }
 
     public ArtifactResolutionException( String message, Throwable cause )
@@ -71,5 +111,10 @@
     public String getType()
     {
         return type;
+    }
+
+    public List getRemoteRepositories()
+    {
+        return remoteRepositories;
     }
 }

Modified: maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolver.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolver.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolver.java (original)
+++ maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolver.java Tue May 10 19:13:18 2005
@@ -18,6 +18,7 @@
 
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.metadata.ArtifactMetadataSource;
+import org.apache.maven.artifact.metadata.ArtifactMetadataRetrievalException;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
 

Modified: maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java (original)
+++ maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java Tue May 10 19:13:18 2005
@@ -100,7 +100,7 @@
             }
             catch ( ArtifactMetadataRetrievalException e )
             {
-                throw new ArtifactResolutionException( e.getMessage(), e );
+                throw new ArtifactResolutionException( e.getMessage(), artifact, remoteRepositories, e );
             }
         }
 
@@ -128,49 +128,19 @@
             }
             catch ( ResourceDoesNotExistException e )
             {
-                throw new ArtifactResolutionException( artifactNotFound( localPath, remoteRepositories ), artifact, e );
+                throw new ArtifactResolutionException( e.getMessage(), artifact, remoteRepositories, e );
             }
             catch ( TransferFailedException e )
             {
-                throw new ArtifactResolutionException( e.getMessage(), artifact, e );
+                throw new ArtifactResolutionException( e.getMessage(), artifact, remoteRepositories, e );
             }
             catch ( ArtifactMetadataRetrievalException e )
             {
-                throw new ArtifactResolutionException( "Error downloading artifact " + artifact, e );
+                throw new ArtifactResolutionException( e.getMessage(), artifact, remoteRepositories, e );
             }
         }
     }
 
-    private static final String LS = System.getProperty( "line.separator" );
-
-    private String artifactNotFound( String path, List remoteRepositories )
-    {
-        StringBuffer sb = new StringBuffer();
-
-        sb.append( "The artifact is not present locally as:" );
-        sb.append( LS );
-        sb.append( LS );
-        sb.append( path );
-        sb.append( LS );
-        sb.append( LS );
-        sb.append( "or in any of the specified remote repositories:" );
-        sb.append( LS );
-        sb.append( LS );
-
-        for ( Iterator i = remoteRepositories.iterator(); i.hasNext(); )
-        {
-            ArtifactRepository remoteRepository = (ArtifactRepository) i.next();
-
-            sb.append( remoteRepository.getUrl() );
-            if ( i.hasNext() )
-            {
-                sb.append( ", " );
-            }
-        }
-
-        return sb.toString();
-    }
-
     // ----------------------------------------------------------------------
     // Transitive modes
     // ----------------------------------------------------------------------
@@ -182,14 +152,7 @@
     {
         ArtifactResolutionResult artifactResolutionResult;
 
-        try
-        {
-            artifactResolutionResult = collect( artifacts, localRepository, remoteRepositories, source, filter );
-        }
-        catch ( TransitiveArtifactResolutionException e )
-        {
-            throw new ArtifactResolutionException( "Error transitively resolving artifacts: ", e );
-        }
+        artifactResolutionResult = collect( artifacts, localRepository, remoteRepositories, source, filter );
 
         for ( Iterator i = artifactResolutionResult.getArtifacts().values().iterator(); i.hasNext(); )
         {
@@ -223,7 +186,7 @@
     private ArtifactResolutionResult collect( Set artifacts, ArtifactRepository localRepository,
                                               List remoteRepositories, ArtifactMetadataSource source,
                                               ArtifactFilter filter )
-        throws TransitiveArtifactResolutionException
+        throws ArtifactResolutionException
     {
         ArtifactResolutionResult result = new ArtifactResolutionResult();
 
@@ -303,8 +266,8 @@
                     }
                     catch ( ArtifactMetadataRetrievalException e )
                     {
-                        throw new TransitiveArtifactResolutionException( "Error retrieving metadata [" + newArtifact +
-                                                                         "] : ", e );
+                        throw new TransitiveArtifactResolutionException( e.getMessage(), newArtifact,
+                                                                         remoteRepositories, e );
                     }
 
                     // the pom for given dependency exisit we will add it to the

Modified: maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/TransitiveArtifactResolutionException.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/TransitiveArtifactResolutionException.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/TransitiveArtifactResolutionException.java (original)
+++ maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/TransitiveArtifactResolutionException.java Tue May 10 19:13:18 2005
@@ -1,5 +1,9 @@
 package org.apache.maven.artifact.resolver;
 
+import org.apache.maven.artifact.Artifact;
+
+import java.util.List;
+
 /*
  * Copyright 2001-2005 The Apache Software Foundation.
  *
@@ -21,16 +25,11 @@
  * @version $Id$
  */
 public class TransitiveArtifactResolutionException
-    extends Exception
+    extends ArtifactResolutionException
 {
-    public TransitiveArtifactResolutionException( String message )
-    {
-        super( message );
-    }
-
-    public TransitiveArtifactResolutionException( Throwable cause )
+    public TransitiveArtifactResolutionException( String message, Artifact artifact, List remoteRepositories, Throwable t )
     {
-        super( cause );
+        super( message, artifact, remoteRepositories, t );
     }
 
     public TransitiveArtifactResolutionException( String message, Throwable cause )

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java Tue May 10 19:13:18 2005
@@ -128,6 +128,10 @@
         {
             throw new ReactorException( "Error processing projects for the reactor: ", e );
         }
+        catch ( ArtifactResolutionException e )
+        {
+            throw new ReactorException( "Error processing projects for the reactor: ", e );
+        }
 
         try
         {
@@ -169,7 +173,7 @@
     }
 
     private List collectProjects( List files, ArtifactRepository localRepository, boolean recursive )
-        throws ProjectBuildingException, ReactorException, IOException
+        throws ProjectBuildingException, ReactorException, IOException, ArtifactResolutionException
     {
         List projects = new ArrayList( files.size() );
 
@@ -278,7 +282,7 @@
     }
 
     public MavenProject getProject( File pom, ArtifactRepository localRepository )
-        throws ProjectBuildingException
+        throws ProjectBuildingException, ArtifactResolutionException
     {
         if ( pom.exists() )
         {

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java?rev=169557&r1=169556&r2=169557&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 Tue May 10 19:13:18 2005
@@ -252,7 +252,7 @@
                 if ( groupId.equals( e.getGroupId() ) && artifactId.equals( e.getArtifactId() ) &&
                     version.equals( e.getVersion() ) && "maven-plugin".equals( e.getType() ) )
                 {
-                    throw new PluginNotFoundException( groupId, artifactId, version, e );
+                    throw new PluginNotFoundException( e );
                 }
                 else
                 {

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/PluginNotFoundException.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/PluginNotFoundException.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/PluginNotFoundException.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/PluginNotFoundException.java Tue May 10 19:13:18 2005
@@ -27,9 +27,9 @@
 public class PluginNotFoundException
     extends ArtifactResolutionException
 {
-    public PluginNotFoundException( String groupId, String artifactId, String version, ArtifactResolutionException e )
+    public PluginNotFoundException( ArtifactResolutionException e )
     {
-        super( "Mojo could not be found - check that the goal name is correct", groupId, artifactId, version,
-               "maven-plugin", e );
+        super( "Mojo could not be found - check that the goal name is correct", e.getGroupId(), e.getArtifactId(),
+               e.getVersion(), "maven-plugin", e.getRemoteRepositories(), e );
     }
 }

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/ArtifactResolverDiagnoser.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/ArtifactResolverDiagnoser.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/ArtifactResolverDiagnoser.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/usability/ArtifactResolverDiagnoser.java Tue May 10 19:13:18 2005
@@ -18,7 +18,11 @@
 
 import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.artifact.resolver.TransitiveArtifactResolutionException;
+import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.project.ProjectBuildingException;
+
+import java.util.List;
+import java.util.Iterator;
 
 public class ArtifactResolverDiagnoser
     implements ErrorDiagnoser

Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java?rev=169557&r1=169556&r2=169557&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 Tue May 10 19:13:18 2005
@@ -137,13 +137,13 @@
     }
 
     public MavenProject build( File projectDescriptor, ArtifactRepository localRepository )
-        throws ProjectBuildingException
+        throws ProjectBuildingException, ArtifactResolutionException
     {
         return buildFromSourceFile( projectDescriptor, localRepository );
     }
 
     private MavenProject buildFromSourceFile( File projectDescriptor, ArtifactRepository localRepository )
-        throws ProjectBuildingException
+        throws ProjectBuildingException, ArtifactResolutionException
     {
         Model model = readModel( projectDescriptor );
 
@@ -168,7 +168,7 @@
 
     public MavenProject buildFromRepository( Artifact artifact, List remoteArtifactRepositories,
                                              ArtifactRepository localRepository )
-        throws ProjectBuildingException
+        throws ProjectBuildingException, ArtifactResolutionException
     {
 
         Model model = findModelFromRepository( artifact, remoteArtifactRepositories, localRepository );
@@ -178,20 +178,13 @@
 
     private Model findModelFromRepository( Artifact artifact, List remoteArtifactRepositories,
                                            ArtifactRepository localRepository )
-        throws ProjectBuildingException
+        throws ProjectBuildingException, ArtifactResolutionException
     {
         Model model = getCachedModel( artifact.getGroupId(), artifact.getArtifactId(), artifact.getVersion() );
         if ( model == null )
         {
-            try
-            {
-                // TODO: can't assume artifact is a POM
-                artifactResolver.resolve( artifact, remoteArtifactRepositories, localRepository );
-            }
-            catch ( ArtifactResolutionException e )
-            {
-                throw new ProjectBuildingException( "Unable to find artifact: " + artifact.toString(), e );
-            }
+            // TODO: can't assume artifact is a POM
+            artifactResolver.resolve( artifact, remoteArtifactRepositories, localRepository );
 
 //                String path = localRepository.pathOfMetadata( new ProjectArtifactMetadata( artifact, null ) );
 //                File file = new File( localRepository.getBasedir(), path );
@@ -202,7 +195,7 @@
     }
 
     private MavenProject build( String pomLocation, Model model, ArtifactRepository localRepository )
-        throws ProjectBuildingException
+        throws ProjectBuildingException, ArtifactResolutionException
     {
         Model superModel = getSuperModel();
 
@@ -285,7 +278,7 @@
 
     private MavenProject assembleLineage( Model model, LinkedList lineage, List aggregatedRemoteWagonRepositories,
                                           ArtifactRepository localRepository )
-        throws ProjectBuildingException
+        throws ProjectBuildingException, ArtifactResolutionException
     {
         aggregatedRemoteWagonRepositories.addAll( buildArtifactRepositories( model.getRepositories() ) );
 

Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProjectBuilder.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProjectBuilder.java?rev=169557&r1=169556&r2=169557&view=diff
==============================================================================
--- maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProjectBuilder.java (original)
+++ maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/MavenProjectBuilder.java Tue May 10 19:13:18 2005
@@ -36,7 +36,7 @@
     static final String STANDALONE_SUPERPOM_VERSION = "2.0";
 
     MavenProject build( File project, ArtifactRepository localRepository )
-        throws ProjectBuildingException;
+        throws ProjectBuildingException, ArtifactResolutionException;
 
     MavenProject buildWithDependencies( File project, ArtifactRepository localRepository,
                                         ArtifactMetadataSource artifactMetadataSource )
@@ -53,7 +53,7 @@
      */
     MavenProject buildFromRepository( Artifact artifact, List remoteArtifactRepositories,
                                       ArtifactRepository localRepository )
-        throws ProjectBuildingException;
+        throws ProjectBuildingException, ArtifactResolutionException;
 
     MavenProject buildStandaloneSuperProject( ArtifactRepository localRepository )
         throws ProjectBuildingException;

Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java?rev=169557&r1=169556&r2=169557&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 Tue May 10 19:13:18 2005
@@ -74,7 +74,7 @@
     }
 
     public Set retrieve( Artifact artifact, ArtifactRepository localRepository, List remoteRepositories )
-        throws ArtifactMetadataRetrievalException
+        throws ArtifactMetadataRetrievalException, ArtifactResolutionException
     {
         // TODO: only metadata is really needed - resolve as metadata
         artifact = artifactFactory.createArtifact( artifact.getGroupId(), artifact.getArtifactId(),
@@ -103,14 +103,7 @@
             // need to be able to not have a project builder
             // TODO: remove - which then makes this a very thin wrapper around a project builder - is it needed?
 
-            try
-            {
-                artifactResolver.resolve( artifact, remoteRepositories, localRepository );
-            }
-            catch ( ArtifactResolutionException e )
-            {
-                throw new ArtifactMetadataRetrievalException( "Error while resolving metadata artifact", e );
-            }
+            artifactResolver.resolve( artifact, remoteRepositories, localRepository );
 
             FileReader reader = null;
             try



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org