On Fri, Jan 31, 2025 at 10:19:29AM -0500, Tom Lane wrote: > FWIW, it seems weird to me to use the word "output" with only one > of those terms. The proposed new wording is fine by me, or we > could do > > "Start a subprocess to capture stderr, csvlog and/or jsonlog output > into log files." > > I read that as 'stderr, csvlog and/or jsonlog' all being modifiers > for 'output', which is sensible grammar.
FWIW, I'd choose for keeping things simpler and just remove the word "output", keeping this list as all the possible values. Hence, the patch is OK here: - gettext_noop("Start a subprocess to capture stderr output and/or csvlogs into log files."), + gettext_noop("Start a subprocess to capture stderr, csvlog and/or jsonlog into log files."), -- Michael
signature.asc
Description: PGP signature