D26993: [libkworkspace] Introduce UpdateLaunchEnvJob

[David Edmundson]

davidedmundson added a comment.


  +++
  
  Lets port some usages, so we can confirm it works in practice, but this looks good

INLINE COMMENTS

> updatelaunchenvjob.cpp:79
> +
> +    // Startup (effectively KSMServer)
> +    org::kde::Startup startup(QStringLiteral("org.kde.Startup"),

Not quite. It's plasma-session

It spawns processes, which in turn might update the launch env, that we should have that env when we spawn the next round of processes.

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma
Cc: davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200129/67f0b018/attachment.html>