On 28 March 2013 07:40, Patrick Hunt <ph...@apache.org> wrote: > On Wed, Mar 27, 2013 at 2:59 PM, sebb <seb...@gmail.com> wrote: >> On 27 March 2013 20:57, Matthieu Morel <mmo...@apache.org> wrote: >>> On Mar 27, 2013, at 21:00 , sebb wrote: >>>> On 27 March 2013 19:07, Matthieu Morel <mmo...@apache.org> wrote: >>> gradle/gradlew scripts to not have the ASL header because this is generated >>> code. >>> >>> According to the RAT tool, generated code does not need to bear the license >>> header. >> >> The RAT tool is just a tool, it does not make the rules. >> >>> This issue was identified and discussed during the voting process on s4-dev >>> mailing lis. For this release, it was considered valid to leave those >>> generated files not annotated by one of our mentors. >>> >> >> That may not have been the correct decision. >> > > Hi Sebb, I had originally -1'd the release due to the missing header > on these generated files. I was going by my understanding of Apache > guidelines, in particular > http://incubator.apache.org/guides/releasemanagement.html#notes-license-headers > and I generally follow that first paragraph - where ever possible add > the header. > > I changed my vote based on the RAT output and then consulting the > second paragraph in that "on license headers" section linked above, in > particular "Copyright may not subsist in a document which is generated > by an transformation from an original." > > What's the correct decision here?
If in doubt, add the AL header. > Patrick > > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
