From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | samay sharma <smilingsamay(at)gmail(dot)com> |
Cc: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, John Naylor <john(dot)naylor(at)enterprisedb(dot)com>, pgsql-hackers(at)postgresql(dot)org, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> |
Subject: | Re: Documentation for building with meson |
Date: | 2022-11-23 20:16:20 |
Message-ID: | 20221123201620.GK11463@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Nov 23, 2022 at 11:30:54AM -0800, samay sharma wrote:
> Thanks for the feedback. Addressed all and added markup at a few more
> places in v6 (attached).
Thanks. It looks good to me. A couple thoughts, maybe they're not
important.
- LZ4 and Zstd refer to wal_compression and default_toast_compression,
but not to any corresponding option to basebackup.
- There's no space after the hash mark here; but above, there was:
#Setup build directory with a different installation prefix
- You use slash to show enumerated options, but it's more typical to
use braces: {a | b | c}:
-Dnls=auto/enabled/disabled
- There's no earlier description/definition of an "auto" feature, but
still says this:
"Setting this option allows you to override value of all 'auto' features"
- Currently the documentation always refers to "PostgreSQL", but you
added two references to "Postgres":
+ If a program required to build Postgres...
+ Once Postgres is built...
--
Justin
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Verite | 2022-11-23 20:18:57 | Re: Tests for psql \g and \o |
Previous Message | Mark Dilger | 2022-11-23 20:11:24 | Re: fixing CREATEROLE |