New dependency for kdelibs and kdebase: dbusmenu-qt

[Alexander Neundorf]

On Saturday 08 May 2010, Modestas Vainius wrote:
> Hello,
>
> On šeštadienis 08 Gegužė 2010 10:07:06 Alexander Neundorf wrote:
> > > So those CMAKE_INSTALL_RPATH are what I had to patch out mostly.
> >
> > Are you aware of other places ?
> > In trunk/KDE/ this *should* be the only place where the RPATH is set.
>
> I don't have trunk checkouts. But as far as I can tell, most cases present
> in KDE 4.4 are still valid in trunk:
>
> akonadi fixed in trunk;
>
> http://websvn.kde.org/trunk/KDE/kdelibs/kdewidgets/CMakeLists.txt?view=mark
>up (look for INSTALL_RPATH);
>
> http://websvn.kde.org/trunk/KDE/kdebase/runtime/phonon/platform_kde/CMakeLi
>sts.txt?view=markup (look for INSTALL_RPATH);
>
> http://gitorious.org/phonon/phonon/blobs/master/CMakeLists.txt (look for
> CMAKE_INSTALL_RPATH);
>
> http://websvn.kde.org/trunk/kdesupport/soprano/CMakeLists.txt?view=markup
> (look for CMAKE_INSTALL_RPATH);
>
> http://websvn.kde.org/trunk/kdesupport/attica/lib/CMakeLists.txt?view=marku
>p (look for CMAKE_INSTALL_RPATH);

Could you just go there and commit the fixes for the packages in kdesupport ?

Not sure why there is the special handling in kdelibs/kdewidgets/ and 
kdebase/runtime/phonon/platform_kde/. I would assume that this is just not 
necessary.
Could you ask the respectibe persons why this is there so ?

Sending an email to Matthias Kretz would probably also be a good idea, I think 
he should know why the code in Phonon is as it is.

Thanks
Alex