GitLab

We no longer install to ksysguard/ we now use ksystemstats/

The names are still not good because that is what we get from libsensors.
But it now also handles minimium and maximum if possible.
To not show the same sensors that we already handle in other plugins,
I use a simple blacklist for now for the ones we want to exclude.

It should no longer be necessary

Rather than ksysguard_ since they are no longer related to ksysguard

Since ksysguard_global doesn't make much sense anymore

We now use the interface name from the systemstats lib

Gets rid of one more reference to ksysguard

LGPL should no longer be needed

All C++ code should now be using `GPL-2.1-only OR GPL-3.0-only OR
LicenseRef-KDE-Accepted-GPL` so we do not have an odd mix of licenses
anymore. Everyone with copyright statements in the files was OK with
this.

Makes things simpler when developing since there's no need to constantly
restart ksystemstats.

Use ksystemstats instead of ksysguard

Use the same string as their non-percentage versions since the % is
somewhat redundant anyway.

Since it adds no information we should not add it in the first place. To
do this, we first need to parse cpuinfo and count the number of physical
CPUs and only after that create the actual objects.

It leads to names like "All Total Usage" which is redundant.

BUG: 433517

It worked because it was the only one with an empty providerName.

The chareg rate is measured in of course in Watts.

Unfortunately the statistics refresh rate of a device is a global property. So
when it is changed from the outside we just force it back to one second. The
initial rate is saved and updated when something tries to change it in order
to restore it upon destruction.
Because NetworkmanagerQt just forwards the DBus signal we have to guard the
slot with a boolean to we do not trigger it ourselves.
BUG:433063
FIXED-IN:5.21.1

These should be 'read' and 'write' instead of readRate and 'writeRate'.
This matches the old sensor names and what the applet expects.
BUG:432851

BUG:430676
FIXED-IN:5.21.0

Classical case of off-by-one. The documentation starts numbering at one.
I used the numbers from it instead of the proper ones.
BUG:432637

This makes it easier to build against both Qt5 and Qt6

GIT_SILENT

In order to read the file it not only does need to exist but we also have to open it.
This broke some systems where a file existed but was not readable with current permission.
If we check if we could open a file we can save the check if the file exists.

We are searching for a StorageDrive to see if it is on a HardDisk or or some
other type of device. However on some setups a Volume will not have StorageDrive
as an ancestor, this is for example the case when using lvm. So let's include
every volume by default and exclude only those for which we find a Drive that is
not a hard disk.

Fix copy and paste mistake
GIT_SILENT

In certain cases, NetworkManager can go on a rampage sending active
connection change signals in very rapid succession. This results in
memory usage ballooning because lambda connections can not be unique.

Rather than destroying and recreating the NetworkManagerDevice objects
on active connection change, this changes things so that we only remove
them from the SensorContainer but keep the actual objects around. This
allows NetworkManagerDevice to track its active connection state,
removing the need for constant reconnects.

BUG: 430003

Otherwise clients looking for the interface will receive an error when the name
is announced but there is not the wanted interface.

We rely on the prefixes on the front page of plasma-systemmonitor; spare every
subclass from having to do it.