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

Martin Gräßlin mgraesslin at kde.org
Mon Jun 16 11:30:15 UTC 2014



> On June 14, 2014, 9:12 p.m., Hrvoje Senjan wrote:
> > startkde/ksyncdbusenv/CMakeLists.txt, line 5
> > <https://git.reviewboard.kde.org/r/118730/diff/2/?file=281040#file281040line5>
> >
> >     it was my understanding that KF5_LIBEXEC_INSTALL_DIR is 'reserved' for KF5, and rest of depending software would use libexec_install_dir.
> >     iow, this is only executable in kf5/ subdir, that doesn't come from a Framework
> 
> Martin Gräßlin wrote:
>     thanks for noticing, I'll fix it later today.

addressed in http://commits.kde.org/plasma-workspace/2e4ec651a5a12e8d46d1afd17b614aa290306ee0


- Martin


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


On June 14, 2014, 10:41 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118730/
> -----------------------------------------------------------
> 
> (Updated June 14, 2014, 10:41 a.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/startkde.cmake 0c1f5841eeefc1012dbdc71e1d1fde11e7511dd6 
>   startkde/ksyncdbusenv/ksyncdbusenv.cpp PRE-CREATION 
>   startkde/ksyncdbusenv/CMakeLists.txt PRE-CREATION 
>   startkde/CMakeLists.txt 3360a1c59409588ecec49e63276914b69568164e 
> 
> 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/20140616/6df438c7/attachment-0001.html>


More information about the Plasma-devel mailing list