sounds good for me too to have a separate rat profile (need to be activated on ci server)
2012/10/25 Benson Margulies <[email protected]>: > 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] > -- Olivier Lamy Talend: http://coders.talend.com http://twitter.com/olamy | http://linkedin.com/in/olamy --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
