Differential D28624

Print a warning if runner is incompatible with KRunner
ClosedPublic
Actions

Authored by davidre on Apr 6 2020, 2:31 PM.

Details

Reviewers
broulik
Commits
R308:e56def21a286: Print a warning if runner is incompatible with KRunner
Summary

Useful for finding out why runner isn't loaded even though it's enabled in
settings. Can for example happen if a runner is installed by an application
that's compiled against a newer KRunner version than Plasma.

Test Plan

Try to load an incompatible runner plugin

Diff Detail

Repository
R308 KRunner
Branch
warning (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24845
Build 24863: arc lint + arc unit
davidre created this revision.Apr 6 2020, 2:31 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 6 2020, 2:31 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
davidre requested review of this revision.Apr 6 2020, 2:31 PM
broulik accepted this revision.Apr 28 2020, 9:12 AM

Would be lovely if we could get this date parsing into QVersionNumber of KFormat, I'm seeing similar code also in kscreenlocker, kwin, libksysguard, ...

This revision is now accepted and ready to land.Apr 28 2020, 9:12 AM
Closed by commit R308:e56def21a286: Print a warning if runner is incompatible with KRunner (authored by davidre). · Explain WhyApr 29 2020, 7:31 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/runnermanager.cpp (7 lines)

Diff 79487

View Options

src/runnermanager.cpp

Loading...