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