[kded] Separate configuration storage path
ClosedPublic

Authored by romangg on Nov 18 2018, 10:12 PM.

Details

Summary

As a preparatory step save the configuration storage directory in a separate
subfolder variable. In KDE6 this should go into a separate subfolder 'config'
of the current data directory.

Test Plan

Autotest updated.

Diff Detail

Repository
R104 KScreen
Branch
1configsLocation
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11596
Build 11614: arc lint + arc unit
romangg created this revision.Nov 18 2018, 10:12 PM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 18 2018, 10:12 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
romangg requested review of this revision.Nov 18 2018, 10:12 PM

Can you provide a kconf update script or have it also check the old locations? Losing all screen config after an upgrade is bad.

romangg added a comment.EditedNov 25 2018, 12:30 PM

Is it fine if we keep the files in the current directory and just add the additional directories outputs and control in the later patches of this series? The hash values should never coincide with these names. So it might be ugly, but work without problems.
Moving the configuration files in a separate configs folder could then be done in KDE6 since with D17007 some of them would then be invalid anyways.

romangg updated this revision to Diff 57659.May 6 2019, 4:37 PM
  • Rebase on master.
  • Prepare changing the configuration path for KDE6.
romangg updated this revision to Diff 57660.May 6 2019, 4:41 PM
  • Fix rebase error
romangg retitled this revision from [kded] Change configuration storage path to [kded] Separate configuration storage path.May 6 2019, 4:45 PM
romangg edited the summary of this revision. (Show Details)
romangg edited the summary of this revision. (Show Details)
romangg updated this revision to Diff 59661.Jun 12 2019, 4:29 PM

Rebase on master.

This revision was not accepted when it landed; it landed in state Needs Review.Jun 13 2019, 9:02 PM
This revision was automatically updated to reflect the committed changes.