Review request: JDK8 b91 source with GPL header errors

David Holmes david.holmes at oracle.com
Tue May 28 20:50:52 PDT 2013


On 29/05/2013 1:33 PM, Mike Duigou wrote:
>
> On May 28 2013, at 18:45 , David Holmes wrote:
>
>> On 29/05/2013 11:35 AM, David Katleman wrote:
>>> Below are corrections to GPL headers found in JDK8 b91
>>>
>>>      http://cr.openjdk.java.net/~katleman/8015525/webrev
>>>
>>> All are punctuation issues, no code is involved, anyone can review.
>>>
>>> As a veteran of many such header reviews, reviewing the patch link at
>>> the top of the webrev is often easier than each individual file.
>>
>> All good.
>>
>> Will we never get to the end of this saga :(
>
> Davids,
>
> I have a partial solution and also the explanation for the missing commas in the lambda test files:
>
> The templates in make/templates don't include a comma after the %YEARS% variable. Can we add one to the templates? These templates are what I have been using and telling others to use. Since I am guilty of pushing some of these without the comma I know, at least, those would have been prevented.

There is no specification for the format of the %YEARS% variable, so 
rather than showing a comma after %YEARS% shouldn't something specify 
that %YEARS% is either:

MMMM,

or

MMMM, NNNN,

else we risk users of the template using eg 2006-2013

David

> HTH,
>
> Mike
>
>>
>> Thanks,
>> David
>>
>>>          Thanks
>>>                  Dave
>


More information about the jdk8-dev mailing list