Re: pg_basebackup: Missing newlines in some error messages - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: pg_basebackup: Missing newlines in some error messages
Date
Msg-id D4773E9C-2E3C-4D61-ACDC-6C7BF14B0E40@yesql.se
Whole thread Raw
In response to pg_basebackup: Missing newlines in some error messages  (Michael Banck <michael.banck@credativ.de>)
Responses Re: pg_basebackup: Missing newlines in some error messages  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
List pgsql-hackers
> On 21 Mar 2018, at 13:12, Michael Banck <michael.banck@credativ.de> wrote:

> while working on something else, I noticed that some error messages in
> pg_basebackup do not have a "\n" at the end, resulting in output like:
>
> |pg_basebackup: could not get COPY data stream: pg_basebackup: removing
> |data directory “data2"

There seems to be a few more in the other files, for example this (and more) in
receivelog.c:

-               fprintf(stderr, _("%s: could not send feedback packet: %s"),
+               fprintf(stderr, _("%s: could not send feedback packet: %s\n"),

Should they get newlines appended as well?

cheers ./daniel

pgsql-hackers by date:

Previous
From: Pavan Deolasee
Date:
Subject: Re: [HACKERS] MERGE SQL Statement for PG11
Next
From: Alvaro Herrera
Date:
Subject: Re: pg_basebackup: Missing newlines in some error messages