[energy KCM] Add battery Technology
ClosedPublic

Authored by meven on Aug 24 2019, 8:27 AM.

Details

Summary

Depends on D23157

Test Plan

Compiles
Tested with KF5 5.62 installed

Diff Detail

Repository
R102 KInfoCenter
Branch
arcpatch-D23392
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 15976
Build 15994: arc lint + arc unit
meven created this revision.Aug 24 2019, 8:27 AM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 24 2019, 8:27 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
meven requested review of this revision.Aug 24 2019, 8:27 AM
meven edited the test plan for this revision. (Show Details)
meven added a comment.Sep 1 2019, 5:40 AM

ping
Possible since D23157

Needs a rebase as it no longer applies cleanly:

$ arc patch D23392
 INFO  Base commit is not in local repository; trying to fetch.
Created and checked out branch arcpatch-D23392.
Branch name arcpatch-D23152 already exists; trying a new name.
Created and checked out branch arcpatch-D23152_1.
Checking patch Modules/energy/package/contents/ui/main.qml...
Checking patch Modules/energy/batterymodel.h...
Checking patch Modules/energy/batterymodel.cpp...
Applied patch Modules/energy/package/contents/ui/main.qml cleanly.
Applied patch Modules/energy/batterymodel.h cleanly.
Applied patch Modules/energy/batterymodel.cpp cleanly.

 Cherry Pick Failed!
 Exception 
Command failed with error #1!
COMMAND
git cherry-pick 'arcpatch-D23152_1'

STDOUT
(empty)

STDERR
error: could not apply 84aed6f... [energy kcm] Display Vendor and model
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

(Run with `--trace` for a full exception trace.)
meven updated this revision to Diff 65188.Sep 2 2019, 4:53 AM

Rebase on master

meven added a comment.Sep 2 2019, 4:56 AM

Please note that the feature needs solid 5.62 installed on system as qml looks in /usr/include/solid/battery.h for the battery model header.
By chance iIf you know how to set the include path used by qml....

Please note that the feature needs solid 5.62 installed on system as qml looks in /usr/include/solid/battery.h for the battery model header.
By chance iIf you know how to set the include path used by qml....

Probably easiest to just wait until 5.62 is tagged in a few days and then commit. This is safe because KInfoCenter is distributed with Plasma, and the next Plasma release (5.17) will have Frameworks 5.62 as the dependency version. I've got a few patches myself that are waiting on 5.62 tagging.

meven planned changes to this revision.Sep 2 2019, 4:06 PM

Waiting for KF 5.62 release

meven requested review of this revision.Oct 2 2019, 9:10 AM

This is now ready to review since KF 5.62 was released.
I have tested it.

ngraham accepted this revision.Oct 2 2019, 2:49 PM

Works perfectly.

This revision is now accepted and ready to land.Oct 2 2019, 2:49 PM
meven edited the test plan for this revision. (Show Details)Oct 2 2019, 5:05 PM
This revision was automatically updated to reflect the committed changes.

What is the difference between "LeadAcid" and "Lead Acid"?

meven added a comment.Oct 3 2019, 6:45 AM

What is the difference between "LeadAcid" and "Lead Acid"?

There is a mistake here, will fix.