[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: New dependency for kdelibs and kdebase: dbusmenu-qt
From:       Alexander Neundorf <neundorf () kde ! org>
Date:       2010-05-11 20:57:08
Message-ID: 201005112257.08397.neundorf () kde ! org
[Download RAW message or body]

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


[prev in list] [next in list] [prev in thread] [next in thread]