+1! Looks good to me. -- Jiang Yan Xu <[email protected]> @xujyan <http://twitter.com/xujyan>
On Thu, May 30, 2013 at 10:31 AM, Brenden Matthews < [email protected]> wrote: > +1! > > > On Thu, May 30, 2013 at 8:04 AM, Luciano Resende <[email protected] > >wrote: > > > On Wed, May 29, 2013 at 9:44 AM, Vinod Kone <[email protected]> wrote: > > > > > Hi Luciano, > > > > > > Thanks for the actionable feedback. Will work on it. > > > > > > Also, had a question about what goes into the NOTICE (and COPYRIGHT) > > file. > > > > > > According to > > > http://apache.org/legal/resolved.html#required-third-party-notices > > > > > > "Apache releases should contain a copy of each license, usually > contained > > > in the LICENSE document. For many licenses this is a sufficient notice. > > For > > > some licenses some additional notice is required. In many cases, this > > will > > > be included within the dependent artifact" > > > > > > > > > I might be reading it wrong, but I thought this meant that we don't > have > > to > > > add 3rd party notices to our NOTICE file, since we are 1) including > their > > > LICENSE in our LICENSE file and 2) including the source tar balls of > 3rd > > > party components and each tar ball has its NOTICE in it. > > > > > > > > Yes, looks like it's acceptable to have the copyright notices together > with > > the license. Just make sure there are nothing on different files, as I > > think I saw a copyright notices in one of the dependencies readme.txt > > > > -- > > Luciano Resende > > http://people.apache.org/~lresende > > http://twitter.com/lresende1975 > > http://lresende.blogspot.com/ > > >
