Please see the diff in https://issues.apache.org/jira/browse/MPOM-38.
It adds a rat profile that anyone can use any time, and also activates it on release. No noisy warnings, just a report in target/ to look at. If no one objects, I'll commit this and start a release on the pom. On Thu, Oct 25, 2012 at 12:12 PM, Kristian Rosenvold <[email protected]> wrote: > I think we should activate such a profile on CI before making if fail > releases. A bit skeptical to > stashing that stuff into the release process since it rightfully belongs in > day to day work and there's enough > to do when releasing.... > > I think a "warnonly" mode would be ok no matter what in the release > profile... > > Kristian > > 2012/10/25 Stephen Connolly <[email protected]> > >> If RAT has a WARN-ONLY mode, lets turn that in the release profile on for >> now. >> >> I don't want to break releases yet, given what I think is the size of the >> problem (being one of the people who has run the RAT reports by hand on a >> number of releases) >> >> When we have the problem down to a more manageable level, then I am fine >> with enforcing. >> >> Lets get the RAT report as part of the project site generated during >> staging so that people can see how big the issue is. >> >> As regards slowing the build... given that its just release builds, adding >> 20-30sec shouldn't be a major issue. >> >> -Stephen >> >> On 25 October 2012 16:36, Benson Margulies <[email protected]> wrote: >> >> > On Thu, Oct 25, 2012 at 10:49 AM, Jason van Zyl <[email protected]> wrote: >> > > It can be a requirement that license headers be intact for releases, >> > that's perfectly reasonable but put it in a profile. Turn on the profile >> > for the CI builds if you like but don't put it in the mainline build. >> > >> > Jason, I hear you, but I'd also like to hear from some of the others >> > who commonly serve as RM's. >> > >> > >> > > >> > > You can fix all the headers using the maven-license-plugin[1] which >> does >> > a good job. >> > > >> > > [1]: http://code.google.com/p/maven-license-plugin/ >> > > >> > > On Oct 25, 2012, at 10:04 AM, Benson Margulies <[email protected]> >> > wrote: >> > > >> > >> I want to turn it on for release builds. >> > >> >> > >> On Thu, Oct 25, 2012 at 9:45 AM, Jason van Zyl <[email protected]> >> wrote: >> > >>> As long as it's in a profile it can't hurt. >> > >>> >> > >>> On Oct 25, 2012, at 9:43 AM, Benson Margulies <[email protected] >> > >> > wrote: >> > >>> >> > >>>> We don't currently run rat in the builds, and I think that it would >> be >> > >>>> prudent to add that to the picture. Anyone object? >> > >>>> >> > >>>> >> --------------------------------------------------------------------- >> > >>>> To unsubscribe, e-mail: [email protected] >> > >>>> For additional commands, e-mail: [email protected] >> > >>>> >> > >>> >> > >>> Thanks, >> > >>> >> > >>> Jason >> > >>> >> > >>> ---------------------------------------------------------- >> > >>> Jason van Zyl >> > >>> Founder & CTO, Sonatype >> > >>> Founder, Apache Maven >> > >>> http://twitter.com/jvanzyl >> > >>> --------------------------------------------------------- >> > >>> >> > >>> happiness is like a butterfly: the more you chase it, the more it >> will >> > >>> elude you, but if you turn your attention to other things, it will >> come >> > >>> and sit softly on your shoulder ... >> > >>> >> > >>> -- Thoreau >> > >>> >> > >>> >> > >>> >> > >>> >> > >>> >> > >> >> > >> --------------------------------------------------------------------- >> > >> To unsubscribe, e-mail: [email protected] >> > >> For additional commands, e-mail: [email protected] >> > >> >> > > >> > > Thanks, >> > > >> > > Jason >> > > >> > > ---------------------------------------------------------- >> > > Jason van Zyl >> > > Founder & CTO, Sonatype >> > > Founder, Apache Maven >> > > http://twitter.com/jvanzyl >> > > --------------------------------------------------------- >> > > >> > > the course of true love never did run smooth ... >> > > >> > > -- Shakespeare >> > > >> > > >> > > >> > > >> > > >> > >> > --------------------------------------------------------------------- >> > To unsubscribe, e-mail: [email protected] >> > For additional commands, e-mail: [email protected] >> > >> > >> --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
