Review Request: Start D-Bus after setting KDE_FULL_SESSION

Oswald Buddenhagen ossi at kde.org
Wed Aug 3 19:19:47 BST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102194/#review5362
-----------------------------------------------------------


this sounds wrong. what if dbus is started earlier, e.g. by a PAM module? this can easily happen once we use the new SecretService.
there should be some interface to push environment variables "into the bus" after it is running. maybe it would be ok to read  a file with VAR=value assignments (one per line, no quoting whatsoever - printenv output) from a defined per-session location before activating services.

- Oswald


On Aug. 3, 2011, 12:15 p.m., Christoph Feck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102194/
> -----------------------------------------------------------
> 
> (Updated Aug. 3, 2011, 12:15 p.m.)
> 
> 
> Review request for kdelibs, Kevin Kofler, George Kiagiadakis, and David Faure.
> 
> 
> Summary
> -------
> 
> As described in bug 267770. Luckily, there is no KDE-specific initialization between D-Bus launch and setting KDE_FULL_SESSION, so interchanging them should not affect KDE itself.
> 
> 
> This addresses bug 267770.
>     http://bugs.kde.org/show_bug.cgi?id=267770
> 
> 
> Diffs
> -----
> 
>   startkde.cmake 92c8753 
> 
> Diff: http://git.reviewboard.kde.org/r/102194/diff
> 
> 
> Testing
> -------
> 
> I don't know "D-Bus activated apps", I assume I don't have any, so I did not see any difference in startup.
> 
> 
> Thanks,
> 
> Christoph
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110803/c7cc7fc4/attachment.htm>


More information about the kde-core-devel mailing list