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 2006/09/12 02:42:17 UTC

svn commit: r442402 - /maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/utils/DependencyStatusSets.java

Author: brianf
Date: Mon Sep 11 17:42:16 2006
New Revision: 442402

URL: http://svn.apache.org/viewvc?view=rev&rev=442402
Log:
Applied patch for [MDEP-24] from Valerio Schiavoni

Modified:
    maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/utils/DependencyStatusSets.java

Modified: maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/utils/DependencyStatusSets.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/utils/DependencyStatusSets.java?view=diff&rev=442402&r1=442401&r2=442402
==============================================================================
--- maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/utils/DependencyStatusSets.java (original)
+++ maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/utils/DependencyStatusSets.java Mon Sep 11 17:42:16 2006
@@ -20,6 +20,8 @@
 
 import java.util.Iterator;
 import java.util.Set;
+import java.util.SortedSet;
+import java.util.TreeSet;
 
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.plugin.logging.Log;
@@ -107,7 +109,9 @@
         }
         else
         {
-            for ( Iterator i = this.resolvedDependencies.iterator(); i.hasNext(); )
+        	SortedSet sortedResolvedDependencies = new TreeSet();
+        	sortedResolvedDependencies.addAll(resolvedDependencies);
+            for ( Iterator i = sortedResolvedDependencies.iterator(); i.hasNext(); )
             {
                 log.info( "   " + ( (Artifact) i.next() ).getId() );
             }
@@ -117,7 +121,9 @@
         {
             log.info( "" );
             log.info( "The following files where skipped: " );
-            for ( Iterator i = this.skippedDependencies.iterator(); i.hasNext(); )
+            SortedSet sortedSkippedDependencies = new TreeSet();
+            sortedSkippedDependencies.addAll(this.skippedDependencies);
+            for ( Iterator i = sortedSkippedDependencies.iterator(); i.hasNext(); )
             {
                 log.info( "   " + ( (Artifact) i.next() ).getId() );
             }
@@ -127,7 +133,9 @@
         if ( this.unResolvedDependencies != null && !this.unResolvedDependencies.isEmpty() )
         {
             log.info( "The following files have NOT been resolved: " );
-            for ( Iterator i = this.unResolvedDependencies.iterator(); i.hasNext(); )
+            SortedSet sortedUnResolvedDependencies = new TreeSet();
+            sortedUnResolvedDependencies.addAll(this.unResolvedDependencies);
+            for ( Iterator i = sortedUnResolvedDependencies.iterator(); i.hasNext(); )
             {
                 log.info( "   " + ( (Artifact) i.next() ).getId() );
             }