D21725: Use a native application for starting plasma

[David Edmundson]

davidedmundson added a comment.


  Concept ++
  
  > Started different systems on different distros, on wayland and x11.
  
  Given this is a huge change, please expand on this and confirm we've covered all paths.
  
  Debug needs cleaning (or QCDebug) before merge.
  
  ----
  
  > #preload the user's locale on first start
  
  is missing
  
  ----

INLINE COMMENTS

> startplasma.cpp:142
> +
> +    //TODO: consider linking directly
> +    const int applyMouseStatus = wayland ? 0 : runSync(QStringLiteral("kapplymousetheme"), { QStringLiteral("kcminputrc_mouse_cursortheme"), QStringLiteral("kcminputrc_mouse_cursorsize") });

Don't yet.

This being here is (I think) the result of a lazy hack due to env vars not getting synced properly from kcminit. I hope to address that.

Obviously your port being a 1:1 clone is fine.

> startplasma.cpp:258
> +    // At this point all environment variables are set, let's send it to the DBus session server to update the activation environment
> +    if (!QStandardPaths::findExecutable(QStringLiteral("dbus-update-activation-environment")).isEmpty())
> +        exitCode = runSync(QStringLiteral("dbus-update-activation-environment"), { QStringLiteral("--systemd"), QStringLiteral("--all") });

braces

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D21725

To: apol, #plasma
Cc: davidedmundson, 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/9148cc32/attachment-0001.html>