Re: [DISCUSS] Adding RAT checks to the default build
LGTM On 8/20/13 6:54 AM, "Chip Childers" wrote: >On Mon, Aug 19, 2013 at 10:19:28PM +, Chiradeep Vittal wrote: >> I think developers will anyway try to optimize their build time. >> This cannot be helped. >> The danger of course is that they 'git push' before running the full >>build. > >Agreed, but I'd at least like to ensure that the full build includes >that check. It's an incremental improvement IMO. Anyone object? > >> >> On 8/19/13 11:39 AM, "Chip Childers" wrote: >> >> >Hey all, >> > >> >Apache RAT 0.10 has been released, improving the speed for a RAT check >> >quite a bit. I've gone ahead and changed the builds.a.o jobs to use >> >version 0.10. >> > >> >I'd like to propose that we add the plugin to our default build, now >> >that it's faster. You can see the speed by running: >> > >> >mvn --projects='org.apache.cloudstack:cloudstack' >> >org.apache.rat:apache-rat-plugin:0.10:check >> > >> >And can compare with 0.8 using: >> > >> >mvn --projects='org.apache.cloudstack:cloudstack' >> >org.apache.rat:apache-rat-plugin:0.8:check >> > >> > >> >This would help us ensure that we don't have licensing issues lingering >> >out on CI servers with little attention. >> > >> >-chip >> >>
Re: [DISCUSS] Adding RAT checks to the default build
Sounds good to me. While devs will always optimize their build, I hope everybody does at least a full build once before committing. Cheers, Hugo Sent from my iPhone On 20 aug. 2013, at 15:54, Chip Childers wrote: > On Mon, Aug 19, 2013 at 10:19:28PM +, Chiradeep Vittal wrote: >> I think developers will anyway try to optimize their build time. >> This cannot be helped. >> The danger of course is that they 'git push' before running the full build. > > Agreed, but I'd at least like to ensure that the full build includes > that check. It's an incremental improvement IMO. Anyone object? > >> >> On 8/19/13 11:39 AM, "Chip Childers" wrote: >> >>> Hey all, >>> >>> Apache RAT 0.10 has been released, improving the speed for a RAT check >>> quite a bit. I've gone ahead and changed the builds.a.o jobs to use >>> version 0.10. >>> >>> I'd like to propose that we add the plugin to our default build, now >>> that it's faster. You can see the speed by running: >>> >>> mvn --projects='org.apache.cloudstack:cloudstack' >>> org.apache.rat:apache-rat-plugin:0.10:check >>> >>> And can compare with 0.8 using: >>> >>> mvn --projects='org.apache.cloudstack:cloudstack' >>> org.apache.rat:apache-rat-plugin:0.8:check >>> >>> >>> This would help us ensure that we don't have licensing issues lingering >>> out on CI servers with little attention. >>> >>> -chip >> >>
Re: [DISCUSS] Adding RAT checks to the default build
On Mon, Aug 19, 2013 at 10:19:28PM +, Chiradeep Vittal wrote: > I think developers will anyway try to optimize their build time. > This cannot be helped. > The danger of course is that they 'git push' before running the full build. Agreed, but I'd at least like to ensure that the full build includes that check. It's an incremental improvement IMO. Anyone object? > > On 8/19/13 11:39 AM, "Chip Childers" wrote: > > >Hey all, > > > >Apache RAT 0.10 has been released, improving the speed for a RAT check > >quite a bit. I've gone ahead and changed the builds.a.o jobs to use > >version 0.10. > > > >I'd like to propose that we add the plugin to our default build, now > >that it's faster. You can see the speed by running: > > > >mvn --projects='org.apache.cloudstack:cloudstack' > >org.apache.rat:apache-rat-plugin:0.10:check > > > >And can compare with 0.8 using: > > > >mvn --projects='org.apache.cloudstack:cloudstack' > >org.apache.rat:apache-rat-plugin:0.8:check > > > > > >This would help us ensure that we don't have licensing issues lingering > >out on CI servers with little attention. > > > >-chip > >
Re: [DISCUSS] Adding RAT checks to the default build
I think developers will anyway try to optimize their build time. This cannot be helped. The danger of course is that they 'git push' before running the full build. On 8/19/13 11:39 AM, "Chip Childers" wrote: >Hey all, > >Apache RAT 0.10 has been released, improving the speed for a RAT check >quite a bit. I've gone ahead and changed the builds.a.o jobs to use >version 0.10. > >I'd like to propose that we add the plugin to our default build, now >that it's faster. You can see the speed by running: > >mvn --projects='org.apache.cloudstack:cloudstack' >org.apache.rat:apache-rat-plugin:0.10:check > >And can compare with 0.8 using: > >mvn --projects='org.apache.cloudstack:cloudstack' >org.apache.rat:apache-rat-plugin:0.8:check > > >This would help us ensure that we don't have licensing issues lingering >out on CI servers with little attention. > >-chip
[DISCUSS] Adding RAT checks to the default build
Hey all, Apache RAT 0.10 has been released, improving the speed for a RAT check quite a bit. I've gone ahead and changed the builds.a.o jobs to use version 0.10. I'd like to propose that we add the plugin to our default build, now that it's faster. You can see the speed by running: mvn --projects='org.apache.cloudstack:cloudstack' org.apache.rat:apache-rat-plugin:0.10:check And can compare with 0.8 using: mvn --projects='org.apache.cloudstack:cloudstack' org.apache.rat:apache-rat-plugin:0.8:check This would help us ensure that we don't have licensing issues lingering out on CI servers with little attention. -chip
