From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, Ian Lawrence Barwick <barwick(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [PATCH] document |
Date: | 2021-08-25 16:39:59 |
Message-ID: | 20210825163959.GY10479@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Aug 25, 2021 at 09:50:13AM -0400, Tom Lane wrote:
> Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> writes:
> > When I applied the patch to the master, I found that the table entries for
> > those function were added into the table for aclitem functions in the docs.
> > I think this is not valid position and needs to be moved to the proper one
> > (maybe the table for system catalog information functions?).
>
> You have to be very careful these days when applying stale patches to
> func.sgml --- there's enough duplicate boilerplate that "patch' can easily
> be fooled into dumping an addition into the wrong place. I doubt that
> the submitter meant the doc addition to go there.
I suppose one solution to this is to use git format-patch -U11 or similar, at
least for doc/
Or write the "duplicate boilerplate" across fewer lines.
And another is to add <!-- function() --> comments before and/or after each.
--
Justin
From | Date | Subject | |
---|---|---|---|
Next Message | John Naylor | 2021-08-25 17:13:08 | Re: badly calculated width of emoji in psql |
Previous Message | Tom Lane | 2021-08-25 15:34:09 | Re: Regression tests for MobilityDB: Continous shutdowns at a random step |