Re: ITS#8504 fix breaks Solaris builds

[quanah@symas.com]

--On Wednesday, June 07, 2017 4:11 PM +0000 quanah@symas.com wrote:

> --On Wednesday, June 07, 2017 3:35 PM +0200 Hallvard Breien Furuseth
> <h.b.furuseth@usit.uio.no> wrote:
>
>> On 06. juni 2017 21:59, quanah@openldap.org wrote:
>>> /usr/include/signal.h:233:12: note: declared here
>>>   extern int sigwait(sigset_t *);
>>>              ^
>>
>> POSIX says it has two arguments:
>>    http://pubs.opengroup.org/onlinepubs/9699919799/functions/sigwait.html
>> Is there a two-argument version of sigwait in the Solaris headers
>> somewhere,
>> maybe enabled by #define POSIX_C_SOURCE 1 or something like that?
>
> It's insanely long....

This patch fixed it on my box:

quanah@sol11-3:~/git/sold-2445/openldap$ git diff
diff --git a/libraries/liblmdb/mdb.c b/libraries/liblmdb/mdb.c
index 8a62eff..1976340 100644
--- a/libraries/liblmdb/mdb.c
+++ b/libraries/liblmdb/mdb.c
@@ -113,6 +113,9 @@ typedef SSIZE_T     ssize_t;
 /* Most platforms have posix_memalign, older may only have memalign */
 #define HAVE_MEMALIGN  1
 #include <malloc.h>
+#if defined (__sun)
+#define _POSIX_PTHREAD_SEMANTICS       1
+#endif
 #endif

 #if !(defined(BYTE_ORDER) || defined(__BYTE_ORDER))


--Quanah

--

Quanah Gibson-Mount
Product Architect
Symas Corporation
Packaged, certified, and supported LDAP solutions powered by OpenLDAP:
<http://www.symas.com>