Bug 23733 - udev rules should use ATTR{} not SYSFS{}
Summary: udev rules should use ATTR{} not SYSFS{}
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: David GEIGER
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-19 06:55 CEST by Curtis Hildebrand
Modified: 2018-10-21 23:42 CEST (History)
1 user (show)

See Also:
Source RPM: displaycal-3.6.2.0-1.mga7.src.rpm
CVE:
Status comment:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Curtis Hildebrand 2018-10-19 06:55:04 CEST 
Description of problem:

I've been seeing this in my logs for a while--just never looked it up till now:

[snip...]
Oct 18 16:45:28 Parkes systemd-udevd[634]: Invalid rule /usr/lib/udev/rules.d/45-Argyll.rules:55: unknown key 'SYSFS{idVendor}'
Oct 18 16:45:28 Parkes systemd-udevd[634]: Invalid rule /usr/lib/udev/rules.d/45-Argyll.rules:58: unknown key 'SYSFS{idVendor}'
Oct 18 16:45:28 Parkes systemd-udevd[634]: Invalid rule /usr/lib/udev/rules.d/45-Argyll.rules:61: unknown key 'SYSFS{idVendor}'
Oct 18 16:45:28 Parkes systemd-udevd[634]: Invalid rule /usr/lib/udev/rules.d/45-Argyll.rules:64: unknown key 'SYSFS{idVendor}'
[snip... more of the same]


The format for udev rules changed from SYSFS{descriptor} to ATTR{descriptor} a long time ago.  See `man udev`.  Basically, the RPM spec should:

sed -i -e 's/SYSFS/ATTR/g' 45-Argyll.rules
Marja Van Waes 2018-10-20 13:43:13 CEST

CC: (none) => marja11
Assignee: bugsquad => geiger.david68210

Comment 1 David GEIGER 2018-10-21 10:58:02 CEST 
Should be fixed in next displaycal 3.7.0.0 using now 55-Argyll.rules for recent udev.
Comment 2 Curtis Hildebrand 2018-10-21 23:42:08 CEST 
Thanks!

Status: NEW => RESOLVED
Resolution: (none) => FIXED
Note You need to log in before you can comment on or make changes to this bug.