I do not know how other distributions feel about this, but I fully expect to be flamed if I suggested to add 2 MB of glib to the Debian base system just to support ConsoleKit's ACLs. The attached patch checks that glib is actually installed before trying to run udev-acl. Also, at boot time glib may be missing anyway since it is in /usr/lib/. IMO udev-acl should be distributed with ConsoleKit, not udev. -- ciao, Marco
--- a/extras/udev-acl/70-acl.rules
+++ b/extras/udev-acl/70-acl.rules
@@ -47,8 +47,16 @@ ENV{ID_SMARTCARD_READER}=="*?", ENV{ACL_
# joysticks
SUBSYSTEM=="input", ENV{ID_CLASS}=="joystick", ENV{ACL_MANAGE}="1"
+LABEL="acl_apply"
+
+ENV{ACL_MANAGE}!="?*", GOTO="acl_end"
+
+# Glib is not declared as a package dependency, so check that it is actually
+# installed before running udev-acl.
+TEST!="/usr/lib/libglib-2.0.so.0", GOTO="acl_end"
+
# apply ACL for all locally logged in users
-LABEL="acl_apply", ENV{ACL_MANAGE}=="?*", TEST=="/var/run/ConsoleKit/database", \
+TEST=="/var/run/ConsoleKit/database", \
RUN+="udev-acl --action=$env{ACTION} --device=$env{DEVNAME}"
LABEL="acl_end"
Attachment:
signature.asc
Description: Digital signature
