[Powerdevil] [Bug 340206] New: do not use Version from org.freedesktop.systemd1.Manager in checkSystemdVersion

[Török Edwin]

https://bugs.kde.org/show_bug.cgi?id=340206

            Bug ID: 340206
           Summary: do not use Version from
                    org.freedesktop.systemd1.Manager in
                    checkSystemdVersion
           Product: Powerdevil
           Version: unspecified
          Platform: Debian unstable
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
          Assignee: plasma-devel at kde.org
          Reporter: edwin+kde at etorok.net

According to the documentation [0] Version is not part of the API and "it
should not be parsed, one may not assume the version to be formatted in any
particular way.".

However checkSystemdVersion() in
./powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp uses Version.

The problem is that systemd-shim doesn't implement version, and this results in
the loss of the Sleep/Hibernate buttons from the KDE menu (see
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=747180).

Although it is rather simple to implement 'Version' in systemd-shim, I don't
know if that should be done given that the documentation says it should not be
used in the first place.

[0] http://www.freedesktop.org/wiki/Software/systemd/dbus/

Reproducible: Always

-- 
You are receiving this mail because:
You are the assignee for the bug.