Re: Log prefix missing for subscriber log messages received from publisher

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Álvaro Herrera <alvherre(at)kurilemu(dot)de>
Cc: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Log prefix missing for subscriber log messages received from publisher
Date: 2025-07-22 11:37:54
Message-ID: CAHGQGwEbxi8CAgmmv5HJSeb68f09LZdOcJ_WT=0NfVf1uaxuFg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jul 22, 2025 at 6:24 PM Álvaro Herrera <alvherre(at)kurilemu(dot)de> wrote:
>
> On 2025-Jul-22, Álvaro Herrera wrote:
>
> > On 2025-07-22, Fujii Masao wrote:
> >
> > > The buildfarm member indri reported the following error, which seems related to
> > > the recent changes in dblink. I'll investigate this later.
> >
> > Ah yes — contrib doesn't have gettext support and macOS doesn't like
> > that. Maybe removing the gettext_noop calls in contrib is the easiest
> > solution. I think another option is to add -lintl to the link line but
> > that's probably messier, and without advantage since we still won't
> > have translation support.
>
> I wonder why doesn't contrib/basic_archive fail though.

Is the issue caused by the use of _() in libpq-be-fe-helpers.h,
instead of using gettext_noop() in dblink.c? At least in my test
environment, the build error disappeared after applying
the following change:

- errmsg_internal("%s: %.*s", _(prefix), len, message));
+ errmsg_internal("%s: %.*s", prefix, len, message));

Regards,

--
Fujii Masao

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ashutosh Bapat 2025-07-22 11:40:26 Re: SQL Property Graph Queries (SQL/PGQ)
Previous Message Nikita Malakhov 2025-07-22 11:24:01 Re: Support for 8-byte TOAST values (aka the TOAST infinite loop problem)