Don't start ksmserver with lockscreen integration on wayland
ClosedPublic

Authored by bshah on Jun 23 2019, 9:25 AM.

Details

Summary

Fixes regression from the startkde C++ re-write which made ksmserver
start the kscreenlocker in wayland session, resulting in the double
lockscreeens.

Test Plan

after applying patch I only get one lockscreen

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bshah created this revision.Jun 23 2019, 9:25 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJun 23 2019, 9:25 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
bshah requested review of this revision.Jun 23 2019, 9:25 AM
apol accepted this revision.Jun 23 2019, 9:32 AM
This revision is now accepted and ready to land.Jun 23 2019, 9:32 AM
This revision was automatically updated to reflect the committed changes.
broulik added inline comments.
startkde/startplasma.h
48

Can you use some flags/enum here, startKSMServer(false) isn't very descript.