Review Request 118730: [startkde] Introduce a ksyncdbusenv helper application

[Martin Gräßlin]

> On June 13, 2014, 3:14 p.m., David Edmundson wrote:
> > I don't understand where this is useful unless you have a broken setup to begin with.
> > 
> > Either you have your XDG paths set up properly for all users at which point you don't need this.
> > Or you have DBus setup to load apps with a missing env, which is clearly a silly thing to do.
> > 
> > This patch will make things to work in the Plasma 5 env; but it'll still be broken if you ran this from other desktops which makes the setup wrong.
> 
> Martin Gräßlin wrote:
>     > I don't understand where this is useful unless you have a broken setup to begin with.
>     
>     I have to disagree. We set quite a lot of stuff in startkde which won't be picked up by DBus, e.g. XCURSOR_THEME. This results in for example the ktp chat window having a different cursor than everything else (I reported this bug to you ;-) ).
> 
> David Edmundson wrote:
>     The cursor thing is where I knew about the updateActivationEnv from.
>     Actually we really need this, as otherwise dbus activated apps will be missing the KDE_FULL_SESSION=true which means it won't load the Qt Platform Theme, which would cause all sorts of problems.
>     
>     So I am starting to agree this might be super useful, but not for the reasons on your G+ post.

> So I am starting to agree this might be super useful, but not for the reasons on your G+ post.

yeah, forget my G+ post for that. Your mention of updateActivationEnv made me look into it in more detail and I realized that there is a huge problem as startkde sets up the whole env but that's not passed on. It used to work back in the days when startkde started the dbus-daemon but that's no more the case.

We will even need more: whenever we change an environment variable it needs to be updated (again the example would be the cursors).


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118730/#review59979
-----------------------------------------------------------


On June 13, 2014, 3:01 p.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118730/
> -----------------------------------------------------------
> 
> (Updated June 13, 2014, 3:01 p.m.)
> 
> 
> Review request for Plasma and David Edmundson.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> [startkde] Introduce a ksyncdbusenv helper application
> 
> This application is invoked by startkde once all environment variables
> are set and syncs them to DBus for the activation environment.
> 
> 
> Diffs
> -----
> 
>   startkde/CMakeLists.txt 3360a1c59409588ecec49e63276914b69568164e 
>   startkde/ksyncdbusenv/CMakeLists.txt PRE-CREATION 
>   startkde/ksyncdbusenv/ksyncdbusenv.cpp PRE-CREATION 
>   startkde/startkde.cmake 0c1f5841eeefc1012dbdc71e1d1fde11e7511dd6 
> 
> Diff: https://git.reviewboard.kde.org/r/118730/diff/
> 
> 
> Testing
> -------
> 
> on log in kglobalacceld5 doesn't pick up the right environment yet, but on manual invocation of the tool + trigger restart of kglobalacceld5 it works. So the approach seems to work, just not yet from startkde (maybe it's too late?)
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140613/bf4a86a5/attachment.html>