[ 
https://issues.apache.org/jira/browse/JCRVLT-239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16268316#comment-16268316
 ] 

Stefan Seifert commented on JCRVLT-239:
---------------------------------------

which is the same behavior as for {{requiresRoot}}, {{allowIndexDefinitions}}, 
{{packageType}} - all those properties are overwritten even if defined in 
properties already. the property is only overwritten if it is set explicitly - 
otherwise the property from properties is preserved.

if you want higher precedence from a value from the properties map for 
"acHandling" i can update the patch. and perhaps add a unit test as well.

> Package Maven Plugin: Add "acHandling" property
> -----------------------------------------------
>
>                 Key: JCRVLT-239
>                 URL: https://issues.apache.org/jira/browse/JCRVLT-239
>             Project: Jackrabbit FileVault
>          Issue Type: Improvement
>          Components: package maven plugin
>    Affects Versions: package-maven-plugin-1.0.0
>            Reporter: Stefan Seifert
>             Fix For: package-maven-plugin-1.0.1
>
>         Attachments: JCRVLT-239-v2.patch, JCRVLT-239.patch
>
>
> currently, it is only possible indirectly via the "properties" property to 
> set the "acHandling" package parameter. furthermore it is not mentioned at 
> all in the plugin docs, nor its possible values.
> as the "acHandling" package property is used quite often in package 
> definitions it would be useful to have a property in the mojo itself, same as 
> it is the case for "requiresRoot", "allowIndexDefinitions", "packageType".



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to