Add kconf update script for ksmserver
ClosedPublic

Authored by davidedmundson on Nov 30 2019, 6:03 PM.

Details

Summary

A config enum value was moved in 506d1807e6a444b6f0fc252b21e8b16b1cf1b2e4 to
kcfg we should port the value for existing setups.

What used to be the string "default" is now "emptySession"
"default" was not the default. (confusingly)

On an upgrade "default" will fail to load and go to the default which is "restorePreviousLogout" and not "emptySession"

Test Plan

Ran script

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Nov 30 2019, 6:03 PM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 30 2019, 6:03 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Nov 30 2019, 6:03 PM

-1 I'd prefer if the config value was kept

-1 I'd prefer if the config value was kept

See D25609

-1 I'd prefer if the config value was kept

Beta is shipped and the value is not kept, so I'd still like this.

Ping.

update copying and pasting something fvogt just did

davidedmundson edited the summary of this revision. (Show Details)Mon, Jan 27, 3:50 PM

[Monday, 27 January 2020] [16:02:16 GMT] <DavidRedondo1> Alright then it looks reasonable to me

Treating that as an approval

This revision was not accepted when it landed; it landed in state Needs Review.Wed, Jan 29, 1:47 PM
This revision was automatically updated to reflect the committed changes.