Re: Pg V10: Patch for bug in bonjour support

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
Cc: Luke Lonergan <luke(at)brightforge(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Pg V10: Patch for bug in bonjour support
Date: 2017-11-08 22:02:59
Message-ID: 20002.1510178579@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> writes:
> On Thu, Nov 9, 2017 at 10:05 AM, Luke Lonergan <luke(at)brightforge(dot)com> wrote:
>> + AC_CHECK_LIB(dns_sd, DNSServiceRefSockFD, [], [AC_MSG_ERROR([library
>> 'dns_sd' is required for Bonjour])])

> It lives in libSystem.dylib (implicitly linked) on macOS, so that
> would break the build there. We'd need something a bit more
> conditional, but I don't know what.

A quick look at the Autoconf manual finds:

`AC_CHECK_LIB' requires some care in usage, and should be avoided
in some common cases. Many standard functions like `gethostbyname'
appear in the standard C library on some hosts, and in special
libraries like `nsl' on other hosts. On some hosts the special
libraries contain variant implementations that you may not want to
use. These days it is normally better to use
`AC_SEARCH_LIBS([gethostbyname], [nsl])' instead of
`AC_CHECK_LIB([nsl], [gethostbyname])'.

If Luke wants to check that that works for him, I can check it on
macOS.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Geoghegan 2017-11-08 22:13:17 Re: Small improvement to compactify_tuples
Previous Message Lucas B 2017-11-08 22:02:30 Re: Early locking option to parallel backup