acpi-support still uses dcop calls

Bug #783095 reported by Luis Silva on 2011-05-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
acpi-support (Ubuntu)
High
Unassigned

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
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:

apport information

tags: added: apport-collected natty
description: updated
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

@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) 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