| From: | "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, "pgsql-docs(at)lists(dot)postgresql(dot)org" <pgsql-docs(at)lists(dot)postgresql(dot)org>, Michael Paquier <michael(at)paquier(dot)xyz> |
| Subject: | RE: pg_type_d.h location incorrect |
| Date: | 2021-05-27 06:13:36 |
| Message-ID: | OS0PR01MB61139DA2FBBBB092F5371265FB239@OS0PR01MB6113.jpnprd01.prod.outlook.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-docs |
On Thursday, May 27, 2021 10:58 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote
>So I ended up with the attached --- what do you think?
The doc-fix LGTM.
But I couldn't apply the patch at HEAD(2941138e60). Maybe you did the fix at a branch other than HEAD?
>Also, looking at this, I note that "~/.pg_service.conf" is still a
>platform-ism. We could make that slightly better by writing
>"$HOME/.pg_service.conf", but is that good enough?
Agreed. Besides, A lot of places in current Doc have been useing ~/.pg_*** already.
IMHO, it's good to keep the consistency.
Regards,
Tang
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Laurenz Albe | 2021-05-27 06:59:53 | Re: [PATCH] fix ICU explorer link in locale documentation |
| Previous Message | Anton Voloshin | 2021-05-27 04:10:50 | Re: [PATCH] fix ICU explorer link in locale documentation |