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] > > > > >
