search

gregkh / usbutils

USB utilities for Linux, including lsusb
http://www.linux-usb.org
356 stars 202 forks source link

lsusb.8.in: do not mention usb.ids #97

Closed aurel32 closed 4 years ago

aurel32 commented 4 years ago

The lsusb binary does not use the usb.ids file anymore, instead it uses the udev hardware database. Mentionning usb.ids in lsusb(8) is therefore very confusing for the users. This patch therefore drops that part.

aurel32 commented 4 years ago

@ValZapod Nope, usb.ids is a different package than usbutils. lsusb in debian does NOT use usb.ids.

aurel32 commented 4 years ago

@aurel32 Try to delete /usr/share/misc/usb.ids and check if lsusb still works. It will not I think...

Well you think wrong, you should actually try that yourself instead of claiming that.

It is not just different it is a must have dependency of https://packages.debian.org/buster/usbutils So you are mistaken.

I am the Debian maintainer of this package, I know what I am talking about. The usbutils package still depends on the usb.ids package as some packages access /usr/share/misc/usb.ids directly and expect that depending on usbutils is enough to get this file on the system. This has to be fixed.

Also read this https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=798148

I advise you to actually read that bug. You will find the list of packages that still depends on usbutils to get the /usr/share/misc/usb.ids file installed. You will also notice that this bug is closed.

In any case, this is NOT the Debian bug tracker, this is the upstream bug tracker. What Debian does is not relevant here, what is relevant is how the upstream non-modified lsusb does. Since usbutils 008 (more precisely commit https://github.com/gregkh/usbutils/commit/c6282a7a0e20fbb9c56837f055843635a43ba8b4), lsusb does not use the usb.ids file. You need to heavily patch it to change that, which Debian does NOT.

gregkh commented 4 years ago

@ValZapod, @aurel32 did nothing wrong here at all, I have no idea why you are arguing...

Anyway, this looks good, will go merge this, thanks for the change.