Hi Brett,

2008/4/12, Brett Porter <[EMAIL PROTECTED]>:
> Vincet,
>
>  I don't see these merged back to trunk?

It is on my TODO list. I need to investigate again to be sure. (see MNG-3518)

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

Same thing :)

Cheers,

Vincent

>
>  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]
>
>

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

Reply via email to