Bruce Korb <bk...@gnu.org> writes:
> On 02/03/13 02:19, Richard Sandiford wrote:
>> Not exactly the most exciting patches, and certainly not worth more than
>> one ping, but:
>> 
>>   fixincludes copyright
>>   http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00442.html
>
> You missed the file header.

That was deliberately in patch 2 though.

> Why bother with dual update issues?

Well, the point is that patch 2 is scripted.  Running the script as-is
would update both the header and version output.  That's actually what
I did in the original patch I posted back in January.  However, Joseph
pointed out that mkheaders should only print the last year rather than a
list of years or a range of years (apparently that's GNU policy):

   http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00320.html

so patch 1 was supposed to be changing that first.

In other words, the copyright= suggestion seemed to be about having
the same string in both the headers and output, which is what you
also get by running the script.  But apparently the copyright notice
in the file header and the copyright notice that we print are supposed
to be different.

>>   http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00443.html
>
> Is the project fixincludes, inclhack, gcc or unnamed?
> I think if you go to the trouble to clean all this up,
> one should be picked and it should be made consistent.  :)

OK, fair enough.  I guess I'll withdraw the fixincludes bits.

Believe me, I don't find copyright year lists vs. ranges any more
interesting than the next man.  This really was supposed to be
an attempt to make things simpler. :-)

Thanks,
Richard

Reply via email to