D21725: Use a native application for starting plasma
Aleix Pol Gonzalez
noreply at phabricator.kde.org
Mon Jun 10 16:23:50 BST 2019
apol created this revision.
apol added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
apol requested review of this revision.
REVISION SUMMARY
At the moment we had several scripts to start the different processes. With this we unify this code into an application that takes care of the whole process.
This allows us to:
- Save on process spawning, we don't need to run a separate process synchronously for every single thing.
- Don't have a redundant configuration file parser but reuse the one we've already optimised in KConfig.
- Issue dbus calls from the process itself instead of spawning qdbus.
- Removes a bunch of duplicated code.
TEST PLAN
Started different systems on different distros, on wayland and x11.
REPOSITORY
R120 Plasma Workspace
BRANCH
apol/startkde
REVISION DETAIL
https://phabricator.kde.org/D21725
AFFECTED FILES
plasma.desktop.cmake
plasmawayland.desktop.cmake
startkde/CMakeLists.txt
startkde/config-startplasma.h.cmake
startkde/kcheckrunning/CMakeLists.txt
startkde/kcheckrunning/kcheckrunning.cpp
startkde/kcheckrunning/kcheckrunning.h
startkde/kstartupconfig/CMakeLists.txt
startkde/kstartupconfig/kdostartupconfig.cpp
startkde/kstartupconfig/kstartupconfig.cpp
startkde/plasma-sourceenv.sh
startkde/startkde.cmake
startkde/startplasma-wayland.cpp
startkde/startplasma-waylandsession.cpp
startkde/startplasma-x11.cpp
startkde/startplasma.cmake
startkde/startplasma.cpp
startkde/startplasma.h
startkde/startplasmacompositor.cmake
To: apol, #plasma
Cc: plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190610/50b18610/attachment.html>
More information about the Plasma-devel
mailing list