acpi-support still uses dcop calls

[Luis Silva]

Binary package hint: acpi-support

kde4 switched from dcop to dbus for inter-process communication. However, some invalid dcop calls are still being made by scripts in acpi-support.

Just do:
> grep dcop /usr/share/acpi-support/*

The result on my machine is:
policy-funcs: (pidof dcopserver > /dev/null && test -x /usr/bin/dcop && /usr/bin/dcop kded kded loadedModules | grep -q klaptopdaemon) ||
screenblank:elif [ `pidof dcopserver` ]; then
screenblank: dcop kdesktop KScreensaverIface lock
---
Architecture: i386
DistroRelease: Ubuntu 11.04
InstallationMedia: Kubuntu 11.04 "Natty Narwhal" - Alpha i386 (20110202)
Package: acpi-support 0.138
PackageArchitecture: i386
ProcEnviron:
 LANGUAGE=
 PATH=(custom, no user)
 LANG=en_GB.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.38-9.43-generic 2.6.38.4
Tags: natty
Uname: Linux 2.6.38-9-generic i686
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups:

[Luis Silva (lacsilva)]

apport information

[Steve Langasek (vorlon)]

If kde has switched away from dcop, doesn't that mean 'pidof dcopserver' is going to return false? In that case this is just backwards-compatibility; how is that a bug?

[Luis Silva (lacsilva)]

@Steve Langasek
Indeed you are right that this is backward-compatibility.
I considered it to be a bug because kde4 has a dbus interface that replaces
the dcop one:
"org.kde.screensaver /ScreenSaver Lock"
The dbus interface is however not contemplated by the script.
Cheers,
--
Luis Silva

[Steve Langasek (vorlon)]

ok, so the issue here is missing support for the new dbus interface, not the dcop query per se. Thanks.