Fujii Masao <masao.fu...@gmail.com> wrote:

> This is because pg_archivecleanup puts the line break "\n" in the head of
> debug message. Why should we do so?
> 
> -----------------------
>  if (debug)
>     fprintf(stderr, "\n%s:  removing \"%s\"", progname, WALFilePath);
> -----------------------

We also need "\n" at line 308.
  L.125: fprintf(stderr, "\n%s:  removing \"%s\"", progname, WALFilePath);
  L.308: fprintf(stderr, "%s:  keep WAL file %s and later", progname, 
exclusiveCleanupFileName);

Note that we don't need a line break at Line 130
because strerror() fills the last %s.
  L.130: fprintf(stderr, "\n%s: ERROR failed to remove \"%s\": %s",

Regards,
---
Takahiro Itagaki
NTT Open Source Software Center



-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to