Differential D2363

Use prefered backend if none is specified
AbandonedPublic
Actions

Authored by cordlandwehr on Aug 6 2016, 4:56 PM.

Details

Reviewers
sebas
Group Reviewers
Plasma
Summary

For out-of-process backends, only the KSCREEN_BACKEND environment
variable is considered while loading. If none is set, fall-back
to the preferred backend.

BUG: 365040

Test Plan

Manual testing. Adding this fallback makes
"kcmshell5 kscreen" show again with working KCM instead of a
"No kscreen backend found..." message.

Diff Detail

Repository
R110 KScreen Library
Branch
Plasma/5.7
Lint
No Linters Available
Unit
No Unit Test Coverage
cordlandwehr updated this revision to Diff 5707.Aug 6 2016, 4:56 PM
cordlandwehr retitled this revision from to Use prefered backend if none is specified.
cordlandwehr updated this object.
cordlandwehr edited the test plan for this revision. (Show Details)
cordlandwehr added reviewers: Plasma, sebas.
Restricted Application added a project: Plasma. · View Herald TranscriptAug 6 2016, 4:56 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
cordlandwehr abandoned this revision.Aug 7 2016, 4:00 PM

I figured out that this "fix" is wrong, since there is already code handling an empty backend name string by falling back to the preferred backend.
Will investigate why that code is not triggered for me.

Revision Contents
Changeset List

PathPackages
Msrc/backendmanager.cpp (5 lines)

Diff 5707

View Options

src/backendmanager.cpp

Loading...