Oooooold link error back again.

[Allen Winter]

On Friday 26 December 2008 7:03:45 am Thiago Macieira wrote:
> Jonas Bähr wrote:
> >Hi,
> >
> >Am 26.12.2008 um 02:41 schrieb Thiago Macieira:
> >> Jonas Bähr wrote:
> >>> 00154160 t __Z24mac_fork_and_reexec_selfv
> >>
> >> That lowercase t means it's not a global symbol.
> >>
> >> Since it's not declared static, I can only conclude that the
> >> KDECORE_EXPORT macro isn't working.
> >>
> >> And reading kdemacros.h.cmake, it seems CMake failed to detect gcc's
> >> support for hidden visibility, so the macros are defined to nothing.
> >> That's your bug #1.
> >>
> >> The bug #2 is that, even though we failed to detect support for the
> >> gcc
> >> attributes, we still compiled with -fvisibility=hidden.
> >>
> >> So, either we have those two bugs (which is unlikely), or something
> >> else
> >> is interfering with kdemacros.h or the KDE_EXPORT macro. Please
> >> check your
> >> kdemacros.h file that was generated and check the preprocessed
> >> output of
> >> the kkernel_mac.cpp compilation.
> >
> >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.