You're right, I will cancel and restart the vote. Arnaud Héritier wrote: > I think you should have restart another vote. > +1 for it. > > On May 25, 2010, at 6:46 PM, Paul Gier wrote: > >> Benjamin Bentmann wrote: >>> Paul Gier wrote: >>> >>>> Staging repo: >>>> https://repository.apache.org/content/repositories/maven-013/ >>> -1 >>> >>> The implementation for MANTRUN-100 has two flaws: >>> >>> if ( classifier == null ) >>> { >>> mavenProject.setFile( file ); >>> } >>> else >>> { >>> ArtifactHandler handler = new DefaultArtifactHandler( type ); >>> Artifact artifact = new AttachedArtifact(mavenProject.getArtifact(), >>> type, classifier, handler); >>> artifact.setFile( file ); >>> artifact.setResolved( true ); >>> log( "Attaching " + file + " as an attached artifact", >>> Project.MSG_VERBOSE ); >>> mavenProject.addAttachedArtifact( artifact ); >>> } >>> >>> a) >>> mavenProject.setFile( file ) sets the path to the POM, not the main >>> artifact. >>> >>> b) >>> DefaultArtifactHandler and AttachedArtifact are internal implementation >>> classes, the code should use MavenProjectHelper and interfaces instead. >>> >>> >>> Benjamin >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org >>> For additional commands, e-mail: dev-h...@maven.apache.org >>> >> Hi Benjamin, >> >> Thanks for the feedback and thanks for fixing MANTRUN-139. >> >> I have fixed a) and b) according to your suggestions, and recreated the >> release. >> The new staging repository is >> >> https://repository.apache.org/content/repositories/maven-014/ >> >> Regarding DefaultArtifactHandler vs. MavenProjectHelper, I was trying to >> find a >> way that the task could be dependent only on the Maven project since the task >> does not have direct access to the Maven components. Maybe there should be a >> way to attach an artifact to a project without requiring the project helper? >> >> >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org >> For additional commands, e-mail: dev-h...@maven.apache.org >> > > Arnaud Héritier > eXo - Software Factory Manager > +33 (0)6 89 76 64 24 > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > For additional commands, e-mail: dev-h...@maven.apache.org >
--------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org