You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Chip Childers <ch...@sungard.com> on 2013/08/19 20:39:03 UTC
[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
Re: [DISCUSS] Adding RAT checks to the default build
Posted by Hugo Trippaers <tr...@gmail.com>.
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 <ch...@sungard.com> wrote:
> On Mon, Aug 19, 2013 at 10:19:28PM +0000, 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" <ch...@sungard.com> 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
Posted by Chiradeep Vittal <Ch...@citrix.com>.
LGTM
On 8/20/13 6:54 AM, "Chip Childers" <ch...@sungard.com> wrote:
>On Mon, Aug 19, 2013 at 10:19:28PM +0000, 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" <ch...@sungard.com> 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
Posted by Chip Childers <ch...@sungard.com>.
On Mon, Aug 19, 2013 at 10:19:28PM +0000, 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" <ch...@sungard.com> 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
Posted by Chiradeep Vittal <Ch...@citrix.com>.
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" <ch...@sungard.com> 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