Vincet,

I don't see these merged back to trunk?

Also, it'd be really helpful to add tests for these changes.

Thanks,
Brett

On 12/04/2008, at 11:33 PM, [EMAIL PROTECTED] wrote:

Author: vsiveton
Date: Sat Apr 12 06:33:37 2008
New Revision: 647446

URL: http://svn.apache.org/viewvc?rev=647446&view=rev
Log:
o sort versions

Modified:
maven/components/branches/maven-2.0.x/maven-artifact/src/main/ java/org/apache/maven/artifact/resolver/DefaultArtifactCollector.java

Modified: maven/components/branches/maven-2.0.x/maven-artifact/src/ main/java/org/apache/maven/artifact/resolver/ DefaultArtifactCollector.java
URL: 
http://svn.apache.org/viewvc/maven/components/branches/maven-2.0.x/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactCollector.java?rev=647446&r1=647445&r2=647446&view=diff
= = = = = = = = ====================================================================== --- maven/components/branches/maven-2.0.x/maven-artifact/src/main/ java/org/apache/maven/artifact/resolver/ DefaultArtifactCollector.java (original) +++ maven/components/branches/maven-2.0.x/maven-artifact/src/main/ java/org/apache/maven/artifact/resolver/ DefaultArtifactCollector.java Sat Apr 12 06:33:37 2008
@@ -347,6 +347,8 @@
artifact.setAvailableVersions( versions );
                                }

+                                Collections.sort( versions );
+
VersionRange versionRange = artifact.getVersionRange();

version = versionRange.matchVersion( versions );



--
Brett Porter
[EMAIL PROTECTED]
http://blogs.exist.com/bporter/


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to