Here you go: https://issues.apache.org/jira/browse/RAT-145
For the record the maven-license-plugin works very well and can insert the correct licenses as well. On Jul 21, 2013, at 5:19 PM, sebb <[email protected]> wrote: > On 21 July 2013 21:23, Jason van Zyl <[email protected]> wrote: >> >> On Jul 21, 2013, at 3:14 PM, sebb <[email protected]> wrote: >> >>> On 21 July 2013 19:47, Jason van Zyl <[email protected]> wrote: >>>> >>>> On Jul 21, 2013, at 2:29 PM, Stephen Connolly >>>> <[email protected]> wrote: >>>> >>>>> Revert my change upping to RAT 0.9 >>>>> >>>>> Stupid plugin has major regression in performance, but 0.8 needs excludes >>>>> for git >>>>> >>>> >>>> Yup, just noticed that as well. After trying to attempt to release my >>>> distribution 4 times and the RAT plugin blowing up 4 times. For the record >>>> that's just the wrong phase to check something in. I don't want to wait >>>> after all my tests to see a license violation report. Additionally it's >>>> running in the release:perform but not anywhere else which is doubly >>>> annoying. So if falling back to 0.8 fixes all that awesome. >>>> >>> >>> AFAIK, the plugin has always bound to the phase verify by default. >>> >> >> Poor choice of phase. Possibly the person who made it got validate and >> verify mixed up. It's not that expensive so do it up-front, but right now it >> runs at the worst of all possible times. Right at the end of the release >> process having it go "Ha ha! I pulled your pants down!" is not very helpful. > > Do you want to file a bug against it? > > https://issues.apache.org/jira/browse/RAT > >>> RAT 0.9 added some extra syntax checks which perform very badly when >>> used on large files without AL headers. >>> RAT 0.8 is also slow on the same large files, but had fewer analysers, >>> so the issue was not so noticeable. >>> So you need to exclude large files that aren't supposed to have >>> headers, e.g. log files, Javadoc. >>> Both run much faster when only given input files with valid AL headers. >>> >>> The performance problem has been fixed for the next release. >>> >>>> Thanks for rolling that back. >>>> >>>>> If I'd had notice I'd have reverted it my self but on a phone so no access >>>>> to revert it... Once they get a proper usable release we *should* be ok... >>>>> Though they don't seem to know how to cut releases with Maven... Like wtf >>>>> is the deal with only -SNAPSHOT docs being public!!! >>> >>> I agree that's wrong, but Creadur is not the only TLP which >>> regenerates site documentation from trunk. >>> Some people argue that it's important to publish the latest docs (even >>> if they apply to code that is not available - I don't subscribe to >>> that) >>> >>> In the case of RAT, I don't think there's much difference - if any - >>> between 0.9 and 0.9-SNAPSHOT docs. >>> >>>>> >>>>> On Sunday, 21 July 2013, Jason van Zyl wrote: >>>>> >>>>>> I just tried to cut a distribution using the existing Maven POM and it >>>>>> let >>>>>> me get through the release:prepare phase without any issues and then >>>>>> failed >>>>>> during the release:perform phase. I have no idea how RAT works, or who >>>>>> set >>>>>> it up but that behavior is sub-optimal. Would probably be all right to be >>>>>> on all the time in the validate phase. Certainly preferable to letting me >>>>>> cut a tag and then blowing up while trying to release. >>>>>> >>>>>> Thanks, >>>>>> >>>>>> Jason >>>>>> >>>>>> ---------------------------------------------------------- >>>>>> Jason van Zyl >>>>>> Founder, Apache Maven >>>>>> http://twitter.com/jvanzyl >>>>>> --------------------------------------------------------- >>>>>> >>>>>> Script timed out >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>> >>>>> -- >>>>> Sent from my phone >>>> >>>> Thanks, >>>> >>>> Jason >>>> >>>> ---------------------------------------------------------- >>>> Jason van Zyl >>>> Founder, Apache Maven >>>> http://twitter.com/jvanzyl >>>> --------------------------------------------------------- >>>> >>>> Script timed out >>>> >>>> >>>> >>>> >>>> >>>> >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: [email protected] >>> For additional commands, e-mail: [email protected] >>> >> >> Thanks, >> >> Jason >> >> ---------------------------------------------------------- >> Jason van Zyl >> Founder, Apache Maven >> http://twitter.com/jvanzyl >> --------------------------------------------------------- >> >> Script timed out >> >> >> >> >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > Thanks, Jason ---------------------------------------------------------- Jason van Zyl Founder, Apache Maven http://twitter.com/jvanzyl --------------------------------------------------------- Script timed out
