Review Request 126022: [ksmserver] Add a dedicated --no-lockscreen command line option

Bhushan Shah bhush94 at gmail.com
Sun Nov 15 10:37:48 UTC 2015


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

Ship it!


Ship It!

- Bhushan Shah


On Nov. 11, 2015, 2:40 p.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126022/
> -----------------------------------------------------------
> 
> (Updated Nov. 11, 2015, 2:40 p.m.)
> 
> 
> Review request for Plasma and Bhushan Shah.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> On Wayland ksmserver should not be responsible for the lockscreen,
> but the compositor takes care of providing the lock screen. To prevent
> conflicts KSMServer gets a new command line option to enable it.
> 
> The code is slightly changed to not pass a series of boolean flags to
> KSMServer, but a flags type.
> 
> The startplasma script for Wayland is adjusted to start ksmserver with
> the new command line option.
> 
> 
> Diffs
> -----
> 
>   ksmserver/main.cpp b1f1b68fe41b86da42d7bea2c088db6d89f0d844 
>   ksmserver/server.h 87e46cbad70913b83f976ea2938606042c653c69 
>   ksmserver/server.cpp ffb15d230b927c5ee28b7196904e0061a14b2d09 
>   startkde/startplasma.cmake 4fab3b1597fee40bf69551af895d5248955eadea 
> 
> Diff: https://git.reviewboard.kde.org/r/126022/diff/
> 
> 
> Testing
> -------
> 
> Started a Plasma/Wayland session with the change.
> 
> * ksmserver is started with --no-lockscreen
> * Using loginctl lock-session gives me one lock window, instead of two as before
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

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


More information about the Plasma-devel mailing list