FreakTheMighty / linux-track

Automatically exported from code.google.com/p/linux-track
MIT License
0 stars 0 forks source link

SYSFS udev rule is deprecated #21

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What is the expected output? What do you see instead?

Expected is no warning message.

Actual is a warning message: 

udevd[1468]: SYSFS{}= will be removed in a future udev version, please use 
ATTR{}= to match the event device, or ATTRS{}= to match a parent device, in 
/lib/udev/rules.d/51-TIR.rules:1

What version of the product are you using? On what operating system?

Gentoo
udev-171

Please provide any additional information below.

I modified the udev rule so it no longer uses the deprecated SYSFS.  Also I 
modified the permission level group to use video for better security.  The 
video group seemed the best fit since that's what this is device is used for.

Index: src/51-TIR.rules
===================================================================
--- src/51-TIR.rules    (revision 571)
+++ src/51-TIR.rules    (working copy)
@@ -1,2 +1 @@
-SUBSYSTEM=="usb", ENV{DEVTYPE}=="usb_device",SYSFS{idVendor}=="131d" , 
MODE="0666"
-
+SUBSYSTEM=="usb", ENV{DEVTYPE}=="usb_device", ATTR{idVendor}=="131d", 
MODE="0660", GROUP="video"

Original issue reported on code.google.com by matt...@gmail.com on 11 Apr 2012 at 1:42

GoogleCodeExporter commented 9 years ago
Thank you for spotting that...
I already received a modified file from Bill (caused problems on Ubuntu 12.04), 
and I'll add the file to the repository today.
Kind regards,

Michal

Original comment by f.jo...@email.cz on 11 Apr 2012 at 6:01

GoogleCodeExporter commented 9 years ago
Fixed in revision 572.
I'm closing the issue now, feel free to reopen if it doesn't work for you...
Kind regards,

Michal

Original comment by f.jo...@email.cz on 12 Apr 2012 at 6:08