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

Korrawit Pruegsanusak detective.conan.1412 at gmail.com
Mon Jan 30 09:12:24 PST 2012


Hello Fridrich,

On Mon, Jan 30, 2012 at 20:52, Fridrich Strba
<fridrich.strba at graduateinstitute.ch> wrote:
> The patch is good-ish, nevertheless, the place to fix it is actually in the
> makefile.mk where one just extends for the purpose of that BUILD_ACTION the
> SOLARINC variable adding this directory.
>
> I fixed it now in master in order to make it consistent with what is done in
> the linux case.

Thanks for that :-) But seems you haven't push it yet ...

> But yes, your patch is correct and most appreciated. Sorry for fixing it
> differently :(

For the sake of consistency, it's all fine. :-)
I just don't know I should fix it in makefile.mk.

Best Regards,
-- 
Korrawit Pruegsanusak


More information about the LibreOffice mailing list