kmyunittest.shell doesn't use local libs

[Alexander Neundorf]

On Wednesday 05 December 2007, Thiago Macieira wrote:
> David Faure wrote:
> >On Tuesday 04 December 2007, Thiago Macieira wrote:
> >> David Faure wrote:
> >> >objdump -p kdirmodeltest | grep PATH
> >> >  RPATH      
> >> > /d/kde/inst/kde4/lib:/d/kde/src/4/qt-copy/lib:/d/kde/inst/kde4/lib
> >> >
> >> >And now I can't remember: since RPATH has more priority over
> >> > LD_LIBRARY_PATH, how could those .shell wrapper scripts ever work at
> >> > all?
> >>
> >> RPATH does *not* have priority over LD_LIBRARY_PATH. RPATH overrides
> >> it.
> >
> >Well that's what I meant :)
> >
> :-)
>
> Those shell scripts were meant to work only under RUNPATH-enabled systems.

... and if you build KDE with RPATH disabled.

> If you don't enable it, they don't work. Like I said, blame your
> distribution.
>
> >> You need RUNPATH for that to work. And your grep shows you don't have
> >> it. Blame your distribution for not enabling a 9-year-old flag in your
> >> linker.
> >
> >You mean inside ld itself?
>
> Or in gcc's specs. That's how I turn it on by default on my system. It's
> easy to do in any installation and doesn't require recompilation of
> anything.

How exactly ?

> >It's inside a "if linux and gcc" block already. I guess the only
> > question is if anyone could have an old system where this wouldn't be
> > supported, but if "9 years" is correct then I guess such linux systems
> > would have other trouble with compiling kde4 anyway, right? (like gcc
> > itself too old).
>
> Agreed.
>
> But this is again a modification of the global CMake flags. We should only
> be applying this to KDE builds. But, since we haven't done those changes
> either, your patch won't hurt now.

I have the feeling that you see some issue here. Which ?

Alex