[jira] [Commented] (MPOM-158) Use some sensible defaults with Apache RAT

2018-01-13 Thread JIRA 

[ 
https://issues.apache.org/jira/browse/MPOM-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16325377#comment-16325377
 ] 

Hervé Boutemy commented on MPOM-158:


default excludes are on by default with rat 0.12 
http://creadur.apache.org/rat/apache-rat-plugin/check-mojo.html#useDefaultExcludes

do we really need to configure anything in Apache parent pom?

> Use some sensible defaults with Apache RAT
> --
>
> Key: MPOM-158
> URL: https://issues.apache.org/jira/browse/MPOM-158
> Project: Maven POMs
>  Issue Type: Improvement
>  Components: asf
>Reporter: John D. Ament
> Attachments: RAT-CONFIG.patch
>
>
> The rat section in the parent pom doesn't include RAT defaults.  Would be 
> good to use some sensible defaults.  



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

[jira] [Commented] (MPOM-158) Use some sensible defaults with Apache RAT

2017-01-10 Thread John D. Ament (JIRA) 

[ 
https://issues.apache.org/jira/browse/MPOM-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15816799#comment-15816799
 ] 

John D. Ament commented on MPOM-158:


Its primarily based on podling input - simplifying the default setup.  I agree, 
and using a flag like {{useDefaultExcludes}} may make more sense in the scheme 
of what every ASF project could use.  Looking at my own local, *.log is only in 
one project so I suspect its a local project issue there.

So, agree to add {{useDefaultExcludes}}?

> Use some sensible defaults with Apache RAT
> --
>
> Key: MPOM-158
> URL: https://issues.apache.org/jira/browse/MPOM-158
> Project: Maven POMs
>  Issue Type: Improvement
>  Components: asf
>Reporter: John D. Ament
> Attachments: RAT-CONFIG.patch
>
>
> The rat section in the parent pom doesn't include RAT defaults.  Would be 
> good to use some sensible defaults.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (MPOM-158) Use some sensible defaults with Apache RAT

2017-01-08 Thread Karl Heinz Marbaise (JIRA) 

[ 
https://issues.apache.org/jira/browse/MPOM-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15810966#comment-15810966
 ] 

Karl Heinz Marbaise commented on MPOM-158:
--

First thanks for offering a patch, but there are already a number of default 
excludes via the the configuration of the 
[apache-rat-plugin|https://creadur.apache.org/rat/apache-rat-plugin/check-mojo.html]
 like 
[useEclipseDefaultExcludea|https://creadur.apache.org/rat/apache-rat-plugin/check-mojo.html#useEclipseDefaultExcludes],
 
[useMavenDefaultExcludes|http://creadur.apache.org/rat/apache-rat-plugin/check-mojo.html#useMavenDefaultExcludes]
 and 
[useIdeaDefaultExcludes|http://creadur.apache.org/rat/apache-rat-plugin/check-mojo.html#useIdeaDefaultExcludes].
Apart from that the {{*.log}} is not currently part of any default exclusion 
cause usually a log files does not belong to the project root folder which is 
unique for each project which means the project has to decide to add this 
explicit.
There is one question coming to my mind: Is there a problem arising which you 
like to solve or something which does not work as expected?


> Use some sensible defaults with Apache RAT
> --
>
> Key: MPOM-158
> URL: https://issues.apache.org/jira/browse/MPOM-158
> Project: Maven POMs
>  Issue Type: Improvement
>  Components: asf
>Reporter: John D. Ament
> Attachments: RAT-CONFIG.patch
>
>
> The rat section in the parent pom doesn't include RAT defaults.  Would be 
> good to use some sensible defaults.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)