[Libreoffice] [PATCH] add include directory for ldap.h, fix msvc build

Fridrich Strba fridrich.strba at graduateinstitute.ch
Sat Jan 28 11:19:31 PST 2012


It can use the native Windows API, yes. I ported it to use the mozilla stuff especially so that the Linux build doesn't need to rely on openldap and I set it for Windows too not to have many different code paths.

F.


Sent from Samsung Mobile

Tor Lillqvist <tml at iki.fi> wrote:

> I've attached a patch to fix msvc build. Currently it has problem in
> postgresql with ldap.h not found.

I very much doubt using some random Mozilla header file that happens
to have the same name ldap.h is a good idea. Presumably we would want
to use the built-in Windows LDAP API. Is this PostgreSQL code supposed
to be portable to Windows at all? If not, why bother with it on
WIndows?

--tml
_______________________________________________
LibreOffice mailing list
LibreOffice at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120128/4c67a187/attachment.html>


More information about the LibreOffice mailing list