kdesupport/kdewin32/include/msvc

[Jarosław Staniek]

Christian Ehrlicher said the following, On 2008-01-30 12:15:
>> Von: "Jarosław Staniek" <js at iidea.pl>
>> Christian Ehrlicher said the following, On 2008-01-30 11:27:
>>>> Von: Jaroslaw Staniek
>>>> SVN commit 768534 by staniek:
>>>>
>>>> add a number of HAVE_* flags so we can easier check existence of these
>>>> functions e.g. in strigi
>>>>
>>> That's not good. You'll get conflicts now when anywhere else HAVE_FOO is
>> defined to 1 or something else. Please use a kdewin prefix or something
>> like this. I also don't understand why you need this at all - cmake configure
>> checks are the correct place to check for such things.
>>
>> Configure checks no longer work because functions like strcasecmp() are 
>> defined in kdewin32 and the checks does not link against kdewin32. This is
>> the 
>> case for instance in strigi. Some other checks do not define 
>> WIN32_LEAN_AND_MEAN and we end up without important HAVE_* macros defined
>> too.
>>
>> Perhaps we should add #ifndef HAVE_FOO ?
>>
> Your system is screwed up. strigi was never supposed to use headers from kdewin32 nor link against kdewin32.
> It compiled fine for the last months and I never heard complains from emerge users that strigi is uncompileable. What changed?

I have used emerge with strigi from trunk. Ask Partick how many compile 
problems we've encountered there last night...

-- 
regards / pozdrawiam, Jaroslaw Staniek
  Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on
  Kexi & KOffice (http://www.kexi.pl/en, http://www.koffice.org/kexi)
  KDE Libraries for MS Windows (http://windows.kde.org)