Bug 3654

Summary:	drakx need to be ported to new udev
Product:	Mageia	Reporter:	D Morgan <dmorganec>
Component:	RPM Packages	Assignee:	Olivier Blin <mageia>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Normal	CC:	mageia, mageia, marja11, thierry.vignaud
Version:	Cauldron	Keywords:	NEEDINFO, PATCH
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	drakx-kbd-mouse-x11	CVE:
Status comment:
Attachments:	Patch to use udevadm info to get the serial

Description D Morgan 2011-12-07 09:36:06 CET

I can't update udev for the moment because this would break the installed, usb_id doesn't exist anymore but is a built in command.


usb_id seems at least needed in mouse.pm to generate /dev/input/by-id/xxx

Manuel Hiebel 2011-12-08 16:20:13 CET

CC: (none) => mageia, thierry.vignaud

Thierry Vignaud 2011-12-08 20:06:25 CET

Source RPM: drakx => drakx-net

Comment 1 Thierry Vignaud 2011-12-08 20:11:07 CET

Isn't it a simple matter of running "udevadm test-builtin usb_ids" instead of usb_ids?
Just try a locally build udev-174+ and reports back whether "udevadm test-builtin usb_ids" works or not

Comment 2 Thierry Vignaud 2011-12-10 06:41:09 CET

Dexter? Ping?

Keywords: (none) => NEEDINFO
Component: Installer => RPM Packages
Hardware: i586 => All

Thierry Vignaud 2011-12-17 23:35:17 CET

CC: (none) => mageia

Comment 3 Colin Guthrie 2011-12-18 02:28:48 CET

test-builtin is an invalid command apparently. Are you sure that's the right syntax?

Comment 4 Colin Guthrie 2011-12-18 03:02:15 CET

Arg... I had older udev installed for testing and forgot :s

I can run this (singular usb_id, not usb_ids plural as in your example - I presume that was a typo):
[colin@jimmy ~]$ udevadm test-builtin usb_id /devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.0/input/input9/mouse1
ID_VENDOR=Microsoft
ID_VENDOR_ENC=Microsoft
ID_VENDOR_ID=045e
ID_MODEL=Microsoft_Optical_Mouse_with_Tilt_Wheel
ID_MODEL_ENC=Microsoft\x20Optical\x20Mouse\x20with\x20Tilt\x20Wheel
ID_MODEL_ID=00d1
ID_REVISION=0120
ID_SERIAL=Microsoft_Microsoft_Optical_Mouse_with_Tilt_Wheel
ID_TYPE=hid
ID_BUS=usb
ID_USB_INTERFACES=:030102:
ID_USB_INTERFACE_NUM=00
ID_USB_DRIVER=usbhid


But, this returns likely better info:
[colin@jimmy ~]$ udevadm info --query=symlink --path=/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.0/input/input9/mouse1 | cut -d' ' -f1
input/by-id/usb-Microsoft_Microsoft_Optical_Mouse_with_Tilt_Wheel-mouse

Thus we can use this easily... will test a patch.

Comment 5 Colin Guthrie 2011-12-18 03:45:09 CET

Created attachment 1262 [details]
Patch to use udevadm info to get the serial

This is a patch that basically replaces the current use of usb_id command with a udevadm command.

I would have preferred to use a variation on the previous command syntax I posted but it gets the path to the mouse dev node, not the event dev node, but as I can't easily test this, I just replaced things and used the serial number as before.

My perl-fu is weak so please review and I'll commit it and push a new update and push new udev.

Thierry Vignaud 2011-12-18 04:25:42 CET

Keywords: (none) => PATCH
Assignee: bugsquad => mageia

Comment 6 D Morgan 2011-12-18 18:43:57 CET

if you want to commit, new udev is on cauldron core/release

Thierry Vignaud 2011-12-18 23:05:02 CET

Source RPM: drakx-net => drakx-kbd-mouse-x11

Comment 7 Colin Guthrie 2011-12-18 23:29:46 CET

New version of drak-kbd-mouse-x11 now pushed.

Comment 8 Colin Guthrie 2011-12-19 11:35:55 CET

OK, so I think this bug can be closed, but it would be nice if Blino could test (I'm assuming you have the necessary h/w?)

Comment 9 Marja Van Waes 2012-01-21 14:08:47 CET

(In reply to comment #8)
> OK, so I think this bug can be closed, but it would be nice if Blino could test
> (I'm assuming you have the necessary h/w?)

@ Blino
Ping?

CC: (none) => marja11

Comment 10 Thierry Vignaud 2012-01-21 15:12:04 CET

Fixed long time ago in 0.99

Status: NEW => RESOLVED
Resolution: (none) => FIXED