Oooooold link error back again.

O illogical1 at gmail.com
Sat Dec 27 02:03:45 CET 2008


On Fri, Dec 26, 2008 at 3:18 PM, Jonas Bähr <jonas.baehr at web.de> wrote:
> Am 26.12.2008 um 20:05 schrieb O:
>
>>>>>
>>>>> Thanks for these pointers! I was able to locate and fix the
>>>>> problem.
>>>>> It was caused by an
>>>>> #ifdef Q_OS_MACX
>>>>> *before* the inclusion of QtGlobals, which defines Q_OS_MACX,
>>>>> therefore the header
>>>>> with the KDE_EXPORT macros was always empty. Patch attached
>>>>> (against
>>>>> kdelibs-4.1.85).
>>>>
>>>> Patch is good.
>>>>
>>>> Can someone apply it? I'm away from my KDE checkout.
>>>
>>> Done. r901662.
>>> _______________________________________________
>>> Kde-buildsystem mailing list
>>> Kde-buildsystem at kde.org
>>> https://mail.kde.org/mailman/listinfo/kde-buildsystem
>>>
>> The patch also removes #include <CoreFoundation/CFString.h> from
>> kkernel_mac.h.
>> Is that ok?
>
> Yes. It's only needed in the implementation. It's true that it has
> nothing to do with the bug though, it's just a clean-up.
>
awesome, thanks for figuring all that out guys.

-- 

P. J. O'Rourke  - "Cleanliness becomes more important when godliness
is unlikely."


More information about the Kde-buildsystem mailing list