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

2015-11-16 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126022/ --- (Updated Nov. 16, 2015, 8:14 a.m.) Status -- This change has been

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

2015-11-15 Thread Bhushan Shah
--- 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

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

2015-11-15 Thread Bhushan Shah
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126022/#review88377 --- startkde/startplasma.cmake (line 35)

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

2015-11-15 Thread Martin Gräßlin
> On Nov. 15, 2015, 11:36 a.m., Bhushan Shah wrote: > > startkde/startplasma.cmake, line 35 > > > > > > Please adjust kwin_wayland call to use this environment variable, i.e > > pass --lockscreen to kwin_wayland

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

2015-11-15 Thread Bhushan Shah
> On Nov. 15, 2015, 4:06 p.m., Bhushan Shah wrote: > > startkde/startplasma.cmake, line 35 > > > > > > Please adjust kwin_wayland call to use this environment variable, i.e > > pass --lockscreen to kwin_wayland

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

2015-11-11 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126022/ --- Review request for Plasma and Bhushan Shah. Repository: plasma-workspace

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

2015-11-11 Thread Bhushan Shah
I kinda don't like idea of the command line option.. For instance if someone uses this option on X11 they are lift without lockscreen.. Any other solution possible? Ideally ksmserver should just retire in Wayland world imo and other functionality of it should be split out of it.. On Nov 11, 2015

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

2015-11-11 Thread Martin Graesslin
On Wednesday, November 11, 2015 3:45:53 PM CET Bhushan Shah wrote: > I kinda don't like idea of the command line option.. For instance if > someone uses this option on X11 they are lift without lockscreen.. Any > other solution possible? I don't think that is an actual threat. This requires