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

List:       kde-commits
Subject:    Re: KDE/kdebase/workspace
From:       David Faure <faure () kde ! org>
Date:       2009-10-07 9:21:39
Message-ID: 200910071121.40387.faure () kde ! org
[Download RAW message or body]

On Sunday 04 October 2009, Michael Jansen wrote:
> SVN commit 1031099 by mjansen:
> 
> Fix startkde for installations when kdelibs and kdebase/workspace have
> different install prefixes.
> 
> One of the problems less that make me wonder "do we really support
>  installing kdelibs and kdebase/* into different prefixes?"

I don't think we do...

> --- trunk/KDE/kdebase/workspace/startkde.cmake #1031098:1031099
> @@ -384,7 +384,7 @@
> 
>  # We set LD_BIND_NOW to increase the efficiency of kdeinit.
>  # kdeinit unsets this variable before loading applications.
> -LD_BIND_NOW=true @LIBEXEC_INSTALL_DIR@/start_kdeinit_wrapper
>  +kcminit_startup 
>  +LD_BIND_NOW=true
>  @KDE4_LIBEXEC_INSTALL_DIR@/start_kdeinit_wrapper +kcminit_startup if test

This kills an important performance improvement, I believe.
Lubos, any comments?

-- 
David Faure, faure@kde.org, sponsored by Nokia to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
[prev in list] [next in list] [prev in thread] [next in thread]