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

Andrew Bayer commented on JCLOUDS-24:
-------------------------------------

Yeah, I hate 4 space indents in XML. It's just silly and makes things hard to 
read, and isn't that common in POMs. If need be, I can switch it to 4, but I'd 
much prefer switching all the POMs to 2.

I am a strong advocate of declaring plugin/dependency versions in the first 
place they show up in a tree of POMs you control. I prefer not to have to go 
look at the apache parent POM to see what version we're getting, etc. 
Eventually, all our repos' parent POMs should end up inheriting from 
jclouds.git/project/pom.xml, and then we can do all our dependency management 
for dependencies and plugins there, but I wanted to get this done first.
                
> Add apache-rat check to top-level POM
> -------------------------------------
>
>                 Key: JCLOUDS-24
>                 URL: https://issues.apache.org/jira/browse/JCLOUDS-24
>             Project: jclouds
>          Issue Type: Bug
>          Components: jclouds-core
>    Affects Versions: 1.7.0
>            Reporter: Andrew Bayer
>            Assignee: Andrew Bayer
>             Fix For: 1.7.0
>
>         Attachments: JCLOUDS-24.patch, JCLOUDS-24.patch, JCLOUDS-24.patch, 
> JCLOUDS-24.patch, rat.txt
>
>
> We need to add the apache-rat-plugin to our POM, and ideally to our build 
> process, so that we can catch violations before they bite us at release time. 
> That said, adding it now would result in failures, so we also need to deal 
> with those failures.

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

Reply via email to