RE: cvs commit: maven-components/maven-core/src/main/java/org/apa che/maven/artifact/installer DefaultArtifactInstaller.java

[Maczka Michal <mi...@imtf.ch>]

I haven't look at the sources and don't knwow what the problem is but feel
free to fix it if you want :) 

I can do it myself no sooner then Saturday..


Michal

> -----Original Message-----
> From: Emmanuel Venisse [mailto:emmanuel@venisse.net]
> Sent: Friday, July 16, 2004 4:23 PM
> To: Maven 2 Developers List
> Subject: Re: cvs commit:
> maven-components/maven-core/src/main/java/org/apache/maven/art
> ifact/inst
> aller DefaultArtifactInstaller.java
> 
> 
> Michal,
> 
> I think there are a problem in implementation of installer.
> version param of installFile method is never use.
> 
> Emmanuel
> 
> >   Modified:
> maven-core/src/main/java/org/apache/maven/artifact/installer
> >                         DefaultArtifactInstaller.java
> >   Log:
> >   Rollback MavenArtifact setting version
> >
> >   Revision  Changes    Path
> >   1.8       +2 -2
> maven-components/maven-core/src/main/java/org/apache/maven/art
> ifact/installe
> r/DefaultArtifactInstaller.java
> >
> >   Index: DefaultArtifactInstaller.java
> >   
> ===================================================================
> >   RCS file:
> /home/cvs/maven-components/maven-core/src/main/java/org/apache
> /maven/artifac
> t/installer/DefaultArtifactInstaller.java,v
> >   retrieving revision 1.7
> >   retrieving revision 1.8
> >   diff -u -r1.7 -r1.8
> >   --- DefaultArtifactInstaller.java 16 Jul 2004 13:40:00 -0000 1.7
> >   +++ DefaultArtifactInstaller.java 16 Jul 2004 14:14:19 -0000 1.8
> >   @@ -80,7 +80,7 @@
> >
> >            dependency.setArtifactId( project.getArtifactId() );
> >
> >   -        dependency.setVersion( version );
> >   +        dependency.setVersion( project.getVersion() );
> >
> >            dependency.setType( type );
> >
>