Add KSysGuard::Sensors library
Needs ReviewPublic

Authored by ahiemstra on Thu, Mar 19, 3:23 PM.

Details

Reviewers
None
Group Reviewers
Plasma
Summary

This adds a new library for communicating with a new KSysGuard
DBus-based daemon that provides sensors. The daemon will be
posted by d_ed later on.

This new library pretty much maps the different objects from
the daemon to C++ objects.

Test Plan

None yet, needs the new daemon.

Diff Detail

Repository
R111 KSysguard Library
Branch
sensors_lib
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24157
Build 24175: arc lint + arc unit
ahiemstra created this revision.Thu, Mar 19, 3:23 PM
Restricted Application added a project: Plasma. · View Herald TranscriptThu, Mar 19, 3:23 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ahiemstra requested review of this revision.Thu, Mar 19, 3:23 PM
ahiemstra updated this revision to Diff 78020.Thu, Mar 19, 3:48 PM
  • Add some documentation to classes
ahiemstra updated this revision to Diff 78080.Fri, Mar 20, 10:15 AM
  • Properly install the export header
ahiemstra updated this revision to Diff 78275.Mon, Mar 23, 10:47 AM
  • Don't use PUBLIC_HEADERS, it's not well supported on earlier CMake
zzag added a subscriber: zzag.Mon, Mar 23, 11:49 AM
zzag added inline comments.
sensors/Sensor.h
3

Could you please update my last name?

git grep -l 'Vlad Zagorodniy' | xargs sed -i 's/Vlad Zagorodniy/Vlad Zahorodnii/g'
ahiemstra updated this revision to Diff 78372.Tue, Mar 24, 4:04 PM
  • Add some documentation to classes
  • Properly install the export header
  • Don't use PUBLIC_HEADERS, it's not well supported on earlier CMake
  • Rebase, update SensorInfo and Daemon XML to new versions