You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by br...@apache.org on 2005/10/13 14:16:51 UTC

svn commit: r320748 - in /maven/components/trunk: maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java

Author: brett
Date: Thu Oct 13 05:16:43 2005
New Revision: 320748

URL: http://svn.apache.org/viewcvs?rev=320748&view=rev
Log:
PR: MNG-122
finish cleanup of resolution exceptions

Modified:
    maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
    maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java

Modified: maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java?rev=320748&r1=320747&r2=320748&view=diff
==============================================================================
--- maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java (original)
+++ maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java Thu Oct 13 05:16:43 2005
@@ -162,8 +162,9 @@
                         }
                         catch ( IOException e )
                         {
-                            throw new ArtifactResolutionException( "Unable to copy resolved artifact for local use",
-                                                                   artifact, remoteRepositories, e );
+                            throw new ArtifactResolutionException(
+                                "Unable to copy resolved artifact for local use: " + e.getMessage(), artifact,
+                                remoteRepositories, e );
                         }
                     }
                     artifact.setFile( copy );

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=320748&r1=320747&r2=320748&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 Thu Oct 13 05:16:43 2005
@@ -408,7 +408,8 @@
             }
             catch ( ArtifactResolutionException e )
             {
-                throw new ProjectBuildingException( projectId, "Error getting the POM in the repository", e );
+                throw new ProjectBuildingException( projectId, "Error getting POM for '" + projectId +
+                    "' from the repository: " + e.getMessage(), e );
             }
             catch ( ArtifactNotFoundException e )
             {
@@ -419,7 +420,8 @@
                 }
                 else
                 {
-                    throw new ProjectBuildingException( projectId, "POM not found in repository", e );
+                    throw new ProjectBuildingException( projectId, "POM '" + projectId + "' not found in repository: " +
+                        e.getMessage(), e );
                 }
             }
         }