D27457: Move kcminit_startup and kded to plasma-session

Aleix Pol Gonzalez noreply at phabricator.kde.org
Tue Feb 18 14:56:16 GMT 2020


apol added a comment.


  Both the isServiceRegistered and the env changes make sense to me.

INLINE COMMENTS

> startup.cpp:232
>          connect(loginSound, &NotificationThread::finished, loginSound, &NotificationThread::deleteLater);
>          loginSound->start();});
>  

Unrelated change.

> startup.cpp:421
> +    m_process->setArguments(args);
> +    m_process->setProcessEnvironment(env);
> +

Maybe it would make sense to do the merging of envs here? This way we only set when it's necessary.

REPOSITORY
  R120 Plasma Workspace

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

To: apol, #plasma, davidedmundson, fvogt
Cc: fvogt, lbeltrame, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 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/20200218/be50d279/attachment-0001.html>


More information about the Plasma-devel mailing list