On 3/12/2020 11:52 AM, Tom Lane wrote:
> Artur Zakirov <zaartur@gmail.com> writes:
>> I'm not familiar with the patch itself. But I think there is just a lack
>> of the comma here, after ", /tmp" :-)
>
> [ blink... ] There definitely is a comma there in the version of the
> patch that's in the Fedora repo.
Ah, I see. I just saw the version Devrim sent (not sure that it is used
in the Fedora repo):
https://git.postgresql.org/gitweb/?p=pgrpms.git;a=blob;f=rpm/redhat/master/postgresql-13/master/postgresql-13-var-run-socket.patch;h=a0292a80ae219b4c8dc1c2e686a3521f02b4330d;hb=HEAD
And I thought there should be a comma at the end of the line after
concatenation to avoid concatenation with NULL:
DEFAULT_PGSOCKET_DIR ", /tmp"
> {
> {"unix_socket_directories", PGC_POSTMASTER, CONN_AUTH_SETTINGS,
> gettext_noop("Sets the directories where Unix-domain sockets will be created."),
> NULL,
> GUC_SUPERUSER_ONLY
> },
> &Unix_socket_directories,
> #ifdef HAVE_UNIX_SOCKETS
> DEFAULT_PGSOCKET_DIR ", /tmp"
> #else
> "",
> #endif
> NULL, NULL, NULL
> },
--
Artur
