Hi,
I guess I'm a little late to get back on this for the release, but
here it is anyway:
There's issues with the "plain udev" rules as they are being written
now. They use SYSFS, which is deprecated, and should instead be using
ATTR as the udev + policykit + hal do. However, this brings to another
issue, as hal is being removed from at least Ubuntu, so the udev with
policy set of rules cannot be used as they are.
For now, I've opted for using simple udev rules, which I have tweaked
a bit. Please see the attached patch.
There are other changes, including which group is being used to give
permissions. I thought 'plugdev' was more fitting than 'dialout', but
I may be wrong.
Mathieu Trudel-Lapierre
mathieu...@gmail.com
---
--- concordance-0.21.orig/libconcord/gen_udev_support 2010-07-26
22:51:51.723214716 -0400
+++ concordance-0.21/libconcord/gen_udev_support 2010-07-26
22:53:13.314075387 -0400
@@ -15,7 +15,7 @@
#
UDEV_POLICY_TEMPLATE='ATTR{idVendor}=="%s", ATTR{idProduct}=="%s",
SYMLINK+="harmony-%%k"'
-UDEV_NO_POLICY_TEMPLATE='SYSFS{idVendor}=="%s",
SYSFS{idProduct}=="%s", MODE="0660", GROUP="dialout"'
+UDEV_NO_POLICY_TEMPLATE='ATTR{idVendor}=="%s", ATTR{idProduct}=="%s",
SYMLINK+="harmony-%%k", GROUP="plugdev"'
HAL_PRE_TEMPLATE=' <match key="usb_device.vendor_id" int="0x%s">'
HAL_RULE_TEMPLATE=' <match key="usb_device.product_id" int="0x%s">
@@ -66,7 +66,7 @@
# Neat trick so that non-harmony devices don't read through a million rules
SUBSYSTEM=="usb_device", GOTO="harmony_usb_rules"
SUBSYSTEM=="usb", GOTO="harmony_usb_rules"
-BUS!="usb", GOTO="harmony_rules_end"
+SUBSYSTEMS!="usb", GOTO="harmony_rules_end"
GOTO="harmony_rules_end"
LABEL="harmony_usb_rules"
END
From: Mathieu Trudel <mathieu...@gmail.com>
Subject: Fix syntax for udev rules; use ATTR as SYSFS is deprecated.
Last-Update: 2010-07-26
Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=567871
Index: concordance-0.21/libconcord/gen_udev_support
===================================================================
--- concordance-0.21.orig/libconcord/gen_udev_support 2010-07-26 22:51:51.723214716 -0400
+++ concordance-0.21/libconcord/gen_udev_support 2010-07-26 22:53:13.314075387 -0400
@@ -15,7 +15,7 @@
#
UDEV_POLICY_TEMPLATE='ATTR{idVendor}=="%s", ATTR{idProduct}=="%s", SYMLINK+="harmony-%%k"'
-UDEV_NO_POLICY_TEMPLATE='SYSFS{idVendor}=="%s", SYSFS{idProduct}=="%s", MODE="0660", GROUP="dialout"'
+UDEV_NO_POLICY_TEMPLATE='ATTR{idVendor}=="%s", ATTR{idProduct}=="%s", SYMLINK+="harmony-%%k", GROUP="plugdev"'
HAL_PRE_TEMPLATE=' <match key="usb_device.vendor_id" int="0x%s">'
HAL_RULE_TEMPLATE=' <match key="usb_device.product_id" int="0x%s">
@@ -66,7 +66,7 @@
# Neat trick so that non-harmony devices don't read through a million rules
SUBSYSTEM=="usb_device", GOTO="harmony_usb_rules"
SUBSYSTEM=="usb", GOTO="harmony_usb_rules"
-BUS!="usb", GOTO="harmony_rules_end"
+SUBSYSTEMS!="usb", GOTO="harmony_rules_end"
GOTO="harmony_rules_end"
LABEL="harmony_usb_rules"
END
------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share
of $1 Million in cash or HP Products. Visit us here for more details:
http://p.sf.net/sfu/dev2dev-palm
_______________________________________________
concordance-devel mailing list
concordance-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/concordance-devel
