Re: Missing comment block at the top of streamutil.h and receivelog.h

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Missing comment block at the top of streamutil.h and receivelog.h
Date: 2014-08-26 21:10:18
Message-ID: CABUevEzH5hqK1vqsqCK+6p_2RbBJ_-Vg8RLLgtQ=w-707q5Gbg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Aug 26, 2014 at 11:03 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Sat, Aug 23, 2014 at 11:59 PM, Michael Paquier
> <michael(dot)paquier(at)gmail(dot)com> wrote:
>> As mentioned in $subject, the header files in src/bin/pg_basebackup do
>> not have a comment block at the top and do not have any copyright
>> text.
>> Any reason for that? Shouldn't we have something for consistency with
>> the other files like in the patch attached?
>
> Probably that's a good idea, but do we really need Author: tags? I
> know we have those in a few places, but certainly not everywhere, and
> as time goes by they tend to be less accurate reflections of who wrote
> the latest code (as opposed to the original code). Furthermore, every
> time we include them, it tends to increase the demand to add even more
> of them because, hey, everybody likes to be acknowledged.

Given that I'm the one named in it - nah, just drop it.

--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Magnus Hagander 2014-08-26 21:16:44 Re: Switch pg_basebackup to use -X stream instead of -X fetch by default?
Previous Message Robert Haas 2014-08-26 21:03:12 Re: Missing comment block at the top of streamutil.h and receivelog.h