Re: [PATCHES] Use non-deprecated APIs for dynloader/darwin.c - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCHES] Use non-deprecated APIs for dynloader/darwin.c
Date
Msg-id 26187.1160332165@sss.pgh.pa.us
Whole thread Raw
Responses Re: [PATCHES] Use non-deprecated APIs for dynloader/darwin.c  (Chris Campbell <chris@bignerdranch.com>)
List pgsql-hackers
Chris Campbell <chris@bignerdranch.com> writes:
> Mac OS X has included dlopen() and friends etc since Mac OS X 10.3.
> The attached patch switches to using those APIs in src/backend/port/
> dynloader/darwin.c (and passes "make check").

Looks good, but I don't think we want to abandon OSX 10.2 support
just yet.  I'll revise this to use a configure probe for dlopen.

My inclination is to apply this one now, since it only affects OSX
and should be easily testable, but to hold off on your other patch
for portable Bonjour support until 8.3 devel starts.  The portability
implications of that one are unclear, and I don't know how to test it
either, so I think putting it in now is too much risk.

            regards, tom lane

pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [COMMITTERS] pgsql: Add include needed for new
Next
From: Tom Lane
Date:
Subject: Re: [COMMITTERS] pgsql: Add include needed for new getrusage() call.