[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