Re: Copyright in partition.h and partition.c

From: Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Copyright in partition.h and partition.c
Date: 2017-09-06 00:56:13
Message-ID: c8330bd9-b8f4-ae81-b342-96a18d84fd7e@lab.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2017/09/05 21:14, Tom Lane wrote:
> Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> writes:
>> On 2017/09/05 15:48, Etsuro Fujita wrote:
>>> Here is the copyright in partition.h:
>>>
>>>  * Copyright (c) 2007-2017, PostgreSQL Global Development Group
>>>
>>> I think it's reasonable that that matches the copyright in partition.c,
>>> but partition.c has:
>>>
>>>  * Portions Copyright (c) 1996-2017, PostgreSQL Global Development Group
>>>  * Portions Copyright (c) 1994, Regents of the University of California
>>>
>>> Is that intentional?
>
>> No, it's unintentional. The difference may have resulted from copying
>> different files to become partition.h and partition.c, respectively.
>
>> Maybe, we should change both to say 2016-2017?
>
>> I don't know the exact rule for how we determine those years. Is there
>> some rule in place about that? When I look at execParallel.c, which
>> supposedly got introduced into the tree recently, I see 1996-2017. OTOH,
>> the files in contrib/bloom all have 2016-2017.
>
> Our usual practice is to write the copyright like it is in partition.c
> even in new files. This avoids any question about whether any of the
> code was copied-and-pasted from somewhere else in PG. Even if not one
> word in the file can be traced to code that was somewhere else before,
> it seems to me that this is an appropriate thing to do, to give due
> credit to those who came before us.

Agreed.

> In short: we should make partition.h's copyright look like partition.c's
> not vice versa.

Attached patch does that.

Thanks,
Amit

Attachment Content-Type Size
partition-h-copyright.patch text/plain 550 bytes

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2017-09-06 02:07:07 Re: increasing the default WAL segment size
Previous Message Magnus Hagander 2017-09-05 23:25:59 Re: pg_basebackup behavior on non-existent slot