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

Reply via email to