[ 
https://jira.codehaus.org/browse/MINSTALL-50?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=303528#comment-303528
 ] 

Ivan Bondarenko commented on MINSTALL-50:
-----------------------------------------

Want it too. Currently this can be more or less achieved by 
maven-resources-plugin in conjunction with 'pomFile' param of 
maven-install-plugin, but this is very ugly approach.

Btw the same for maven-deploy-plugin. I wonder why these two plugins are not 
children of one common abstract parent - a lot of logic is just duplicated and 
issues are the same.
                
> provide property filtering on .pom files placed in local repo
> -------------------------------------------------------------
>
>                 Key: MINSTALL-50
>                 URL: https://jira.codehaus.org/browse/MINSTALL-50
>             Project: Maven 2.x Install Plugin
>          Issue Type: New Feature
>          Components: install:install
>    Affects Versions: 2.3
>         Environment: independent
>            Reporter: Stefan Armbruster
>         Attachments: MNG-maven-install.patch, MNG-maven-install.patch
>
>
> When maven installs an artifact, it's pom is also copied into the artifact's 
> directory. Unfortunately, if the pom contains a property reference (e.g. 
> ${myprop}), this will not be replaced upon copying the pom file.
> I've created a patch for the install plugin that switches on property 
> filtering by setting a mojo parameter "filteringEnabled". Since this defaults 
> to "false", backward compatibility is kept 100%. 
> Some implementation notes:
> * the dirty work is done in FilteredProjectArtifactMetadata.java, the 
> property resolution code has been inspired by ResourcesMojo.
> * I've added a unit test, that replaces ${basedir} with the value of a system 
> property.
> * since "svn diff" does not handle binary files, 
> src/test/resources/unit/basic-install-test-with-filtering/target/maven-install-test-1.0-SNAPSHOT.jar
>  is not included in the patch. This file is the same as 
> src/test/resources/unit/basic-install-test/target/maven-install-test-1.0-SNAPSHOT.jar
> Since my knowledge of Maven API is more than limited, there might be a more 
> elegant way to provide this feature ... but it works! I'd be happy to see 
> this in a future release of maven.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to