avro-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Matt Massie <m...@cloudera.com>
Subject Re: [VOTE] Avro release 1.0.0 (candidate 0)
Date Thu, 02 Jul 2009 21:19:02 GMT
On Jul 2, 2009, at 2:16 PM, Patrick Hunt wrote:

> Doug Cutting wrote:
>> Patrick Hunt wrote:
>>> * rat tool reports a number of issues with licensing (see  
>>> attached) of source code files
>> Wow, that was a big attachment!
>
> Weird, looks like RAT includes all that by default. I just attached  
> the output of the command. bummer.
>
>> C and C++ sources need license headers added generally.
>
> *.py as well? Any other source type you support?
>
> for example:
> !????? ./src/c++/scripts/gen.py
>
> Are you including licenses/notices for any/all that need it in lib/ 
> *.jar and lib/py/*?
>
> Otw licensing cleanup sounds good to me.
>
> I did notice that you include Makefile in src/c, I'm no expert but  
> typically don't you just include configure script and let the end  
> user run it to generate the Makefile? Not a showstopper regardless.

This Makefile artifact will be fixed with the patch I'm working on now.

-Matt

Mime
View raw message