Hi, On Wed, Nov 23, 2022 at 12:16 PM Justin Pryzby <pry...@telsasoft.com> wrote:
> 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. > Thank you. Attaching v7 addressing most of the points below. > > - 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 > Added a space as that looks better. > > - You use slash to show enumerated options, but it's more typical to > use braces: {a | b | c}: > -Dnls=auto/enabled/disabled > Changed. > > - 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" > Described what an "auto" feature is in (). > > - 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... > Good catch. Changed to PostgreSQL. Regards, Samay > > -- > Justin >
v7-0001-Add-docs-for-building-with-meson.patch
Description: Binary data