Re: Minimum Maven version

2016-07-22 Thread Suneel Marthi
That fixed it Joe, it was those XML parser warnings that were failing RAT. We are indeed good now. Thanks again. On Fri, Jul 22, 2016 at 10:16 PM, Joe Witt wrote: > If you go here >https://travis-ci.org/apache/incubator-pirk > > You can then click on 'build history' >

Re: Minimum Maven version

2016-07-22 Thread Joe Witt
If you go here https://travis-ci.org/apache/incubator-pirk You can then click on 'build history' https://travis-ci.org/apache/incubator-pirk/builds You can then look for the failed/master build which mentions rat https://travis-ci.org/apache/incubator-pirk/builds/145628437 Then click the

Re: Minimum Maven version

2016-07-22 Thread Suneel Marthi
how do i access that? On Fri, Jul 22, 2016 at 10:05 PM, Joe Witt wrote: > Did you look in /home/travis/build/apache/incubator-pirk/target/rat.txt? > > The RAT reports are generally pretty good I have found. > > On Fri, Jul 22, 2016 at 10:04 PM, Suneel Marthi

Re: Minimum Maven version

2016-07-22 Thread Joe Witt
Did you look in /home/travis/build/apache/incubator-pirk/target/rat.txt? The RAT reports are generally pretty good I have found. On Fri, Jul 22, 2016 at 10:04 PM, Suneel Marthi wrote: > Could it be that the .travis.yml needs a license header, that's the only > one

Re: Minimum Maven version

2016-07-22 Thread Suneel Marthi
Could it be that the .travis.yml needs a license header, that's the only one without a header. On Fri, Jul 22, 2016 at 9:41 PM, Joe Witt wrote: > Suneel > > It is probably a good idea to keep it there because it can really help > with automating some of the checks as

Re: Minimum Maven version

2016-07-22 Thread Joe Witt
Suneel It is probably a good idea to keep it there because it can really help with automating some of the checks as people put in PRs. I'm happy to help do licensing/header hunting soon. Thanks Joe On Fri, Jul 22, 2016 at 6:53 PM, Suneel Marthi wrote: > We r good now,

Re: Minimum Maven version

2016-07-22 Thread Suneel Marthi
We r good now, I disabled the rat check to get past the build failure, i guess we don't need that for travis, do we? On Fri, Jul 22, 2016 at 6:19 PM, Suneel Marthi wrote: > Getting a travis build error for RAT, any ideas ? I am not seeing this > locally. > > [ERROR]

Re: Minimum Maven version

2016-07-22 Thread Suneel Marthi
Getting a travis build error for RAT, any ideas ? I am not seeing this locally. [ERROR] Failed to execute goal org.apache.rat:apache-rat-plugin:0.11:check (default) on project pirk: Too many files with unapproved license: 1 See RAT report in:

Re: Minimum Maven version

2016-07-22 Thread Suneel Marthi
... I am also gonna disable Jdk 7 jobs on travis as part of this change. OK with @here ? On Fri, Jul 22, 2016 at 6:01 PM, Suneel Marthi wrote: > Let's go with Option 1, i can modify .travis.yml to account for Maven 3.3x > > On Fri, Jul 22, 2016 at 5:58 PM, Tim