Header comments in the recently added files

On Wed, Mar 9, 2011 at 8:33 PM, Itagaki Takahiro
<itagaki.takahiro@gmail.com> wrote:

I found trivial mistakes in the recently added files.
Will they fixed by some automated batches, or by manual?

- "Copyright (c) xxx-*2010*, PostgreSQL Global Development Group"
 in pg_collation.h, pg_foreign_table.h, basebackup.h, syncrep.h,
 pg_backup_directory.c and auth_delay.c.
- "IDENTIFICATION $PostgreSQL$" in pg_collation.h, syncrep.h, and syncrep.c
 Other files has their actual paths in the same place.

I think these should be fixed manually. The first might eventually
get fixed automatically, but perhaps not until 2012, and I'm not sure
the second would ever get fixed automatically.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On Thu, Mar 10, 2011 at 12:55, Robert Haas <robertmhaas@gmail.com> wrote:

On Wed, Mar 9, 2011 at 8:33 PM, Itagaki Takahiro
<itagaki.takahiro@gmail.com> wrote:

I found trivial mistakes in the recently added files.
Will they fixed by some automated batches, or by manual?

I think these should be fixed manually.  The first might eventually
get fixed automatically, but perhaps not until 2012, and I'm not sure
the second would ever get fixed automatically.

OK, I'll do that.

--
Itagaki Takahiro

Itagaki Takahiro <itagaki.takahiro@gmail.com> writes:

I found trivial mistakes in the recently added files.
Will they fixed by some automated batches, or by manual?

- "Copyright (c) xxx-*2010*, PostgreSQL Global Development Group"
in pg_collation.h, pg_foreign_table.h, basebackup.h, syncrep.h,
pg_backup_directory.c and auth_delay.c.
- "IDENTIFICATION $PostgreSQL$" in pg_collation.h, syncrep.h, and syncrep.c
Other files has their actual paths in the same place.

It might be worth Bruce making another run of his copyright-update
script to fix the former problems. As for the latter problems,
I have a to-do item to go around and standardize the format of the
header comments a bit better --- I'd like to have a uniform convention
that we put the full path name of each file at the top, and then get
rid of the vestigial "IDENTIFICATION" sections. Don't know quite when
I'll get to it though.

regards, tom lane

Tom Lane wrote:

Itagaki Takahiro <itagaki.takahiro@gmail.com> writes:

I found trivial mistakes in the recently added files.
Will they fixed by some automated batches, or by manual?

- "Copyright (c) xxx-*2010*, PostgreSQL Global Development Group"
in pg_collation.h, pg_foreign_table.h, basebackup.h, syncrep.h,
pg_backup_directory.c and auth_delay.c.
- "IDENTIFICATION $PostgreSQL$" in pg_collation.h, syncrep.h, and syncrep.c
Other files has their actual paths in the same place.

It might be worth Bruce making another run of his copyright-update
script to fix the former problems. As for the latter problems,

I ran it just now and nothing was changed, so we are OK now.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +