Re: Exclude pg_internal.init from base backup

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>
Cc: Simon Riggs <simon(at)2ndquadrant(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net>, Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>, David Steele <david(at)pgmasters(dot)net>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Exclude pg_internal.init from base backup
Date: 2017-11-08 00:11:00
Message-ID: CAB7nPqQ=eRqDGa_GPmtOmDSg0BpwkFgj37OF=J17QzbE650T6g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Nov 8, 2017 at 9:04 AM, Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> wrote:
> The commit 98267e missed to check the empty SGML tag, attached patch
> fixes the same.

<listitem>
<para>
- <filename>pg_internal.init (found in multiple directories)</>
+ <filename>pg_internal.init (found in multiple directories)</filename>
</para>
</listitem>
What has been committed in 98267ee and what is proposed here both look
incorrect to me. The markup filename ought to be used only with file
names, so "(found in multiple directories)" should not be within it.
Simon's commit is not wrong with the markup usage by the way.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Nico Williams 2017-11-08 00:45:52 Re: MERGE SQL Statement for PG11
Previous Message Haribabu Kommi 2017-11-08 00:04:51 Re: Exclude pg_internal.init from base backup