acpi-support still uses dcop calls

Bug #783095 reported by Luis Silva
This bug affects 1 person
Affects Status Importance Assigned to Milestone
acpi-support (Ubuntu)

Bug Description

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
 PATH=(custom, no user)
ProcVersionSignature: Ubuntu 2.6.38-9.43-generic
Tags: natty
Uname: Linux 2.6.38-9-generic i686
UpgradeStatus: No upgrade log present (probably fresh install)

Revision history for this message
Luis Silva (lacsilva) wrote : Dependencies.txt

apport information

tags: added: apport-collected natty
description: updated
Revision history for this message
Steve Langasek (vorlon) wrote :

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?

Changed in acpi-support (Ubuntu):
status: New → Incomplete
Revision history for this message
Luis Silva (lacsilva) wrote : Re: [Bug 783095] Re: acpi-support still uses dcop calls

@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.
Luis Silva

Revision history for this message
Steve Langasek (vorlon) wrote :

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

Changed in acpi-support (Ubuntu):
importance: Undecided → High
status: Incomplete → Triaged
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers