On Wednesday 07 October 2009, Lubos Lunak wrote: > On Wednesday 07 of October 2009, David Faure wrote: > > On Sunday 04 October 2009, Michael Jansen wrote: > > > --- 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? > > It just changes the path (to the same thing anyway), I don't see how it > should affect anything.
No, it runs kcminit_startup before kdeinit, so kcminit_startup doesn't benefit from the fork+dlopen and other optimizations done by kdeinit. Ignore the @...@ part, my comment was about the splitting of "kdeinit +kcminit_startup" into two lines. -- David Faure, [email protected], sponsored by Nokia to work on KDE, Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org). _______________________________________________ Kde-buildsystem mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-buildsystem
