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

Herve Boutemy commented on MNG-5720:
------------------------------------

true: this is not necessary for merging
I doubt toolchain configuration will be complex: what more than base directory?

Notice I don't have any problem if you're convinced of the contrary: not 
something absolutely critical, we can stay with DOM for configuration :)

And I like the idea of a toProperties helper: we'll just have to figure out 
where to put it, because it can't be in maven-core for compatibility



> Change 'configuration' from Object to Properties in toolchains.xml
> ------------------------------------------------------------------
>
>                 Key: MNG-5720
>                 URL: https://jira.codehaus.org/browse/MNG-5720
>             Project: Maven
>          Issue Type: Improvement
>          Components: Toolchains
>    Affects Versions: 3.2.3
>            Reporter: Herve Boutemy
>            Assignee: Robert Scholte
>             Fix For: 3.2.4
>
>
> Officially the content of the {{<configuration>}} element in a toolchain was 
> free of structure, although we only know Properties-like structures.
> To improve the automatic handling by Modello, it is better to drop the free 
> structure and change it to Properties like it was done in MNG-5718 for 
> <provides>.
> The {{jdk}} toolchain will be modified accordingly, being transparent for 
> users.
> But any custom toolchain implementation will need to be updated.



--
This message was sent by Atlassian JIRA
(v6.1.6#6162)

Reply via email to