Bug 3654 - drakx need to be ported to new udev
Summary: drakx need to be ported to new udev
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: Olivier Blin
QA Contact:
URL:
Whiteboard:
Keywords: NEEDINFO, PATCH
Depends on:
Blocks:
 
Reported: 2011-12-07 09:36 CET by D Morgan
Modified: 2012-01-21 15:12 CET (History)
4 users (show)

See Also:
Source RPM: drakx-kbd-mouse-x11
CVE:
Status comment:


Attachments
Patch to use udevadm info to get the serial (1.03 KB, patch)
2011-12-18 03:45 CET, Colin Guthrie
Details | Diff

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


Note You need to log in before you can comment on or make changes to this bug.