You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ah...@apache.org on 2006/09/12 15:56:11 UTC

svn commit: r442585 - in /maven/maven-1/core/trunk/src/java/org/apache/maven: repository/AbstractArtifact.java verifier/DependencyVerifier.java

Author: aheritier
Date: Tue Sep 12 06:56:11 2006
New Revision: 442585

URL: http://svn.apache.org/viewvc?view=rev&rev=442585
Log:
Improve messages for invalid dependencies

Modified:
    maven/maven-1/core/trunk/src/java/org/apache/maven/repository/AbstractArtifact.java
    maven/maven-1/core/trunk/src/java/org/apache/maven/verifier/DependencyVerifier.java

Modified: maven/maven-1/core/trunk/src/java/org/apache/maven/repository/AbstractArtifact.java
URL: http://svn.apache.org/viewvc/maven/maven-1/core/trunk/src/java/org/apache/maven/repository/AbstractArtifact.java?view=diff&rev=442585&r1=442584&r2=442585
==============================================================================
--- maven/maven-1/core/trunk/src/java/org/apache/maven/repository/AbstractArtifact.java (original)
+++ maven/maven-1/core/trunk/src/java/org/apache/maven/repository/AbstractArtifact.java Tue Sep 12 06:56:11 2006
@@ -197,4 +197,26 @@
     {
         this.overrideType = overrideType;
     }
+
+    /**
+     * @see java.lang.Object#toString()
+     */
+    public String toString()
+    {
+        StringBuffer result = new StringBuffer();
+        result.append( getName() );
+        if ( dependency != null )
+        {
+            result.append( " ( type = " );
+            result.append( getDependency().getType() );
+            result.append( ", groupid = " );
+            result.append( getDependency().getGroupId() );
+            result.append( ", artifactid = " );
+            result.append( getDependency().getArtifactId() );
+            result.append( ", version = " );
+            result.append( getDependency().getVersion() );
+            result.append( " )" );
+        }
+        return result.toString();
+    }
 }

Modified: maven/maven-1/core/trunk/src/java/org/apache/maven/verifier/DependencyVerifier.java
URL: http://svn.apache.org/viewvc/maven/maven-1/core/trunk/src/java/org/apache/maven/verifier/DependencyVerifier.java?view=diff&rev=442585&r1=442584&r2=442585
==============================================================================
--- maven/maven-1/core/trunk/src/java/org/apache/maven/verifier/DependencyVerifier.java (original)
+++ maven/maven-1/core/trunk/src/java/org/apache/maven/verifier/DependencyVerifier.java Tue Sep 12 06:56:11 2006
@@ -219,7 +219,7 @@
         for ( Iterator i = failedDependencies.iterator(); i.hasNext(); )
         {
             Artifact artifact = (Artifact) i.next();
-            message.append( artifact.getName() );
+            message.append( artifact );
 
             String overrideType = artifact.getOverrideType();
             if ( overrideType != Artifact.OVERRIDE_NONE )
@@ -287,7 +287,7 @@
                 directory.mkdirs();
             }
 
-            LOGGER.info( getMessage( "download.message", artifact.getName() ) );
+            LOGGER.info( getMessage( "download.message", artifact ) );
 
             if ( getRemoteArtifact( artifact ) )
             {
@@ -369,6 +369,8 @@
         for ( Iterator i = context.getMavenRepoRemote().iterator(); i.hasNext(); count++ )
         {
             String remoteRepo = (String) i.next();
+
+            LOGGER.info( "Searching in repository : " + remoteRepo );
 
             Repository repository = new Repository( "repo" + count, remoteRepo.trim() );