> On June 22, 2014, 2:54 p.m., Àlex Fiestas wrote:
> > According to Lennart we should not call TakeControl, so I guess that while 
> > we wait for kdbus we should discard this review?
> 
> Martin Gräßlin wrote:
>     why? And what does that have to do with kdbus? And how long should we 
> wait for kdbus (my understanding here is that we cannot use kdbus any time 
> soon as Qt won't support it)?

Now that I gained a little more insight into the whole situation, I agree that 
TakeControl() is not ksmserver's duty - that's up to X.org/Wayland/...

The reason I was seeing the "The session is not registered with logind: "PID 
2979 does not belong to any known session"" message was caused by the fact, 
that I'm running everything as systemd user-session services, which are 
currently run outside any logind context - they aren't assigned to any 
systemd-logind session.

I think I can still make some progress on the "Run a Plasma Workspace based on 
user-session services" without having to wait on memfd/kdbus as there are some 
"workarounds" on other fronts (e.g. 
http://cgit.freedesktop.org/polkit/commit/?id=a68f5dfd7662767b7b9822090b70bc5bd145c50c)
 which could help getting forward.

I'll discard this RR and keep on experimenting further on all this stuff.


- Elias


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


On June 19, 2014, 7:03 p.m., Elias Probst wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118804/
> -----------------------------------------------------------
> 
> (Updated June 19, 2014, 7:03 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Bugs: 335390
>     https://bugs.kde.org/show_bug.cgi?id=335390
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This is an initial (not yet working) attempt to fix bug#335390.
> 
> After some discussion in #systemd, it seems there's some more work needed 
> outside of ksmserver. Will have to figure out the next actions to be taken to 
> continue on this.
> 
> See also my mail on systemd-devel: 
> http://lists.freedesktop.org/archives/systemd-devel/2014-June/020238.html
> 
> 
> Diffs
> -----
> 
>   ksmserver/screenlocker/logind.cpp dcfc7f321b3cf29ef68aac8006aa37f5e4e00956 
> 
> Diff: https://git.reviewboard.kde.org/r/118804/diff/
> 
> 
> Testing
> -------
> 
> - Copy /usr/lib/systemd/system/systemd-logind.service to 
> /etc/systemd/system/systemd-logind.service
> - Set "Environment=SYSTEMD_LOG_LEVEL=debug" in the [Service] section of 
> /etc/systemd/system/systemd-logind.service
> - Run "systemctl daemon-reload"
> - Reboot (also possible without a reboot, but far more complicated and 
> requires to terminate the X session anyways, so a reboot is the most 
> straightforward solution)
> 
> To test
> - apply patch + rebuild plasma-workspace
> - kill ksmserver
> - Run "journalctl -n 20 -f -u systemd-logind" to monitor logind
> - Run "tail -f ~/.xsession-errors" or "journalctl --user -n 20 -f --user-unit 
> ksmserver" (for systemd user-session users) to monitor ksmserver's output
> - restart ksmserver
> 
> 
> Thanks,
> 
> Elias Probst
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to