You are viewing a plain text version of this content. The canonical link for it is here.
Posted to general@incubator.apache.org by Branko Čibej <br...@xbc.nu> on 2009/11/17 16:48:09 UTC

Re: svn status update (was: svn commit: r880911 [1/13] - in /subversion/trunk: ./ build/ build/generator/ build/win32/ notes/obliterate/ packages/python-windows/ packages/windows-WiX/BuildSubversion/ packages/windows-WiX/BuildSubversion/WixDialog/ pa

Niall Pemberton wrote:
> I ran a RAT report (0.6 version)  on subversion trunk but theres alot
> of *noise* from it. I have opened a bug ticket for license header
> review here:
>
> http://subversion.tigris.org/issues/show_bug.cgi?id=3529
>   

Very helpful, thanks.

> It would make review easier if the files without headers had them
> added. If thats considered a good idea, I'll try and help submitting
> patches.
>   

Is there any way to make RAT slightly smarter so that it doesn't require
license headers in README files and such? Does policy dictate these
files must have license headers, too?

-- Brane


---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: svn status update (was: svn commit: r880911 [1/13] - in /subversion/trunk: ./ build/ build/generator/ build/win32/ notes/obliterate/ packages/python-windows/ packages/windows-WiX/BuildSubversion/ packages/windows-WiX/BuildSubversion/WixDialog/ pa

Posted by Niall Pemberton <ni...@gmail.com>.
On Tue, Nov 17, 2009 at 4:09 PM, Justin Erenkrantz
<ju...@erenkrantz.com> wrote:
> On Tue, Nov 17, 2009 at 3:48 PM, Branko Čibej <br...@xbc.nu> wrote:
>> Is there any way to make RAT slightly smarter so that it doesn't require
>> license headers in README files and such?
>
> http://svn.apache.org/repos/asf/incubator/rat/main/trunk/
>
> *duck*

David Crossley suggested using the Ant task in this thread:

http://markmail.org/message/ocmn3r5x45qcc42d

That works well with the last release of Ant (0.6) - so no need for
trunk - I've attached an ant build file to the subversion ticket:

http://subversion.tigris.org/issues/show_bug.cgi?id=

Niall

>> Does policy dictate these
>> files must have license headers, too?
>
> No.  -- justin
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: svn status update (was: svn commit: r880911 [1/13] - in /subversion/trunk: ./ build/ build/generator/ build/win32/ notes/obliterate/ packages/python-windows/ packages/windows-WiX/BuildSubversion/ packages/windows-WiX/BuildSubversion/WixDialog/ pa

Posted by Justin Erenkrantz <ju...@erenkrantz.com>.
On Tue, Nov 17, 2009 at 3:48 PM, Branko Čibej <br...@xbc.nu> wrote:
> Is there any way to make RAT slightly smarter so that it doesn't require
> license headers in README files and such?

http://svn.apache.org/repos/asf/incubator/rat/main/trunk/

*duck*

> Does policy dictate these
> files must have license headers, too?

No.  -- justin

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: running RAT with excludes (Was: svn status update...)

Posted by Niall Pemberton <ni...@gmail.com>.
On Tue, Nov 17, 2009 at 11:28 PM, David Crossley <cr...@apache.org> wrote:
> Niall Pemberton wrote:
>> David Crossley wrote:
>> > Branko ??ibej wrote:
>> >> Niall Pemberton wrote:
>> >> > I ran a RAT report (0.6 version) ?on subversion trunk but theres alot
>> >> > of *noise* from it. I have opened a bug ticket for license header
>> >> > review here:
>> >> >
>> >> > http://subversion.tigris.org/issues/show_bug.cgi?id=3529
>> >>
>> >> Very helpful, thanks.
>> >>
>> >> > It would make review easier if the files without headers had them
>> >> > added. If thats considered a good idea, I'll try and help submitting
>> >> > patches.
>> >>
>> >> Is there any way to make RAT slightly smarter so that it doesn't require
>> >> license headers in README files and such? Does policy dictate these
>> >> files must have license headers, too?
>> >
>> > We have been using an "excludes" file to run RAT over Forrest,
>> > driven by the Ant task, using both Buildbot and Gump.
>> >
>> > I explained it today to rat-dev, so that should help you
>> > to get started.
>>
>> Thanks, found it:
>> http://apache.markmail.org/message/xp5xqvw4mmb5yhw7
>>
>> I'll take a look. As Justin pointed out RAT trunk has a -e option for
>> excludes - but perhaps would be easier to build up a list in ant.
>
> We did try the "-e" but it was nowhere near powerful enough for
> our needs. The anttask is beaut.

Thanks David - that works great

Niall

> -David

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: running RAT with excludes (Was: svn status update...)

Posted by David Crossley <cr...@apache.org>.
Niall Pemberton wrote:
> David Crossley wrote:
> > Branko ??ibej wrote:
> >> Niall Pemberton wrote:
> >> > I ran a RAT report (0.6 version) ?on subversion trunk but theres alot
> >> > of *noise* from it. I have opened a bug ticket for license header
> >> > review here:
> >> >
> >> > http://subversion.tigris.org/issues/show_bug.cgi?id=3529
> >>
> >> Very helpful, thanks.
> >>
> >> > It would make review easier if the files without headers had them
> >> > added. If thats considered a good idea, I'll try and help submitting
> >> > patches.
> >>
> >> Is there any way to make RAT slightly smarter so that it doesn't require
> >> license headers in README files and such? Does policy dictate these
> >> files must have license headers, too?
> >
> > We have been using an "excludes" file to run RAT over Forrest,
> > driven by the Ant task, using both Buildbot and Gump.
> >
> > I explained it today to rat-dev, so that should help you
> > to get started.
> 
> Thanks, found it:
> http://apache.markmail.org/message/xp5xqvw4mmb5yhw7
> 
> I'll take a look. As Justin pointed out RAT trunk has a -e option for
> excludes - but perhaps would be easier to build up a list in ant.

We did try the "-e" but it was nowhere near powerful enough for
our needs. The anttask is beaut.

-David

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: running RAT with excludes (Was: svn status update...)

Posted by Niall Pemberton <ni...@gmail.com>.
On Tue, Nov 17, 2009 at 11:01 PM, David Crossley <cr...@apache.org> wrote:
> Branko ??ibej wrote:
>> Niall Pemberton wrote:
>> > I ran a RAT report (0.6 version)  on subversion trunk but theres alot
>> > of *noise* from it. I have opened a bug ticket for license header
>> > review here:
>> >
>> > http://subversion.tigris.org/issues/show_bug.cgi?id=3529
>>
>> Very helpful, thanks.
>>
>> > It would make review easier if the files without headers had them
>> > added. If thats considered a good idea, I'll try and help submitting
>> > patches.
>>
>> Is there any way to make RAT slightly smarter so that it doesn't require
>> license headers in README files and such? Does policy dictate these
>> files must have license headers, too?
>
> We have been using an "excludes" file to run RAT over Forrest,
> driven by the Ant task, using both Buildbot and Gump.
>
> I explained it today to rat-dev, so that should help you
> to get started.

Thanks, found it:
http://apache.markmail.org/message/xp5xqvw4mmb5yhw7

I'll take a look. As Justin pointed out RAT trunk has a -e option for
excludes - but perhaps would be easier to build up a list in ant.

Niall

> -David

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


running RAT with excludes (Was: svn status update...)

Posted by David Crossley <cr...@apache.org>.
Branko ??ibej wrote:
> Niall Pemberton wrote:
> > I ran a RAT report (0.6 version)  on subversion trunk but theres alot
> > of *noise* from it. I have opened a bug ticket for license header
> > review here:
> >
> > http://subversion.tigris.org/issues/show_bug.cgi?id=3529
> 
> Very helpful, thanks.
> 
> > It would make review easier if the files without headers had them
> > added. If thats considered a good idea, I'll try and help submitting
> > patches.
> 
> Is there any way to make RAT slightly smarter so that it doesn't require
> license headers in README files and such? Does policy dictate these
> files must have license headers, too?

We have been using an "excludes" file to run RAT over Forrest,
driven by the Ant task, using both Buildbot and Gump.

I explained it today to rat-dev, so that should help you
to get started.

-David

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org