On 2026-Jul-15, Anton Voloshin wrote: > On 08/06/2026 21:03, Álvaro Herrera wrote: > > > --- a/src/include/postmaster/syslogger.h > > +++ b/src/include/postmaster/syslogger.h > > @@ -85,6 +85,7 @@ extern PGDLLIMPORT int syslogPipe[2]; > > extern PGDLLIMPORT HANDLE syslogPipe[2]; > > #endif > > > > +extern bool syslogger_setup_done; > > Shouldn't this be extern *PGDLLIMPORT* bool ... ?
Hmm, does that export the symbol so that third-party code can get at it? I'm not sure it makes sense to do it for this one symbol, which should be internal business between syslogger and elog only. Did you hit a problem with it? If so, what was it? Michael, looking at https://github.com/michaelpq/pg_plugins/blob/main/jsonlog/jsonlog.c do you see an issue with this variable? -- Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/ "La espina, desde que nace, ya pincha" (Proverbio africano)
