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

Korrawit Pruegsanusak <detective.conan.1412 -AT- gmail.com>
Tue, 31 Jan 2012 00:12:24 +0700

Hello Fridrich,

On Mon, Jan 30, 2012 at 20:52, Fridrich Strba
<fridrich.strba@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

Context

Re: [Libreoffice] [PATCH] add include directory for ldap.h, fix msvc build · Fridrich Strba
- Re: [Libreoffice] [PATCH] add include directory for ldap.h, fix msvc build · Korrawit Pruegsanusak
  - Re: [Libreoffice] [FIXED] add include directory for ldap.h, fix msvc build · Fridrich Strba
    - Re: [Libreoffice] [FIXED] add include directory for ldap.h, fix msvc build · Korrawit Pruegsanusak

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.