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

Attachment: signature.asc
Description: PGP signature

Reply via email to