| 1 |
Robin H. Johnson wrote: |
| 2 |
> Heya, |
| 3 |
> |
| 4 |
> So now this is not a flamewar. |
| 5 |
> |
| 6 |
> Jakub was originally going to complain at me for the upstream usbutils |
| 7 |
> adding support for gzipped usb.ids files, but a group of us (myself, |
| 8 |
> dsd, jakub, leio, steev) had a discussion about it, and came up with a |
| 9 |
> solution that both ends the breakage for direct users (HAL and others), |
| 10 |
> and provides forward momentum. |
| 11 |
> |
| 12 |
> So firstly, what's the real problem? The original complaint came up |
| 13 |
> because HAL expected the uncompressed file to exist as pci.ids, and |
| 14 |
> wasn't ready to look at pci.ids.gz. While this caused breakage, it was |
| 15 |
> only a warning sign that there was a deeper problem. |
| 16 |
|
| 17 |
I don't feel strongly enough to make an objection to your commit, but I |
| 18 |
think pciutils is doing the right thing, and despite me and Mike putting |
| 19 |
a hours into getting a decent HAL patch together the response I got was |
| 20 |
that as upstream they are simply "not interested" (no technical or |
| 21 |
logical objections provided), so I don't feel you should be putting |
| 22 |
workarounds in pciutils just to make HAL happy. |
| 23 |
|
| 24 |
Especially because HAL really doesn't use pci.ids for anything useful. I |
| 25 |
am attaching a HAL ebuild patch which is the approach I'm in favour of |
| 26 |
and first mentioned several months ago. It does not require any HAL |
| 27 |
patches or pciutils modifications. It stems from the fact that really |
| 28 |
HAL doesn't really do anything useful with the ID-to-name mappings |
| 29 |
provided in pci.ids. It makes "that HAL bug" disappear with the click of |
| 30 |
the fingers. I didn't really get any proper answer why our HAL |
| 31 |
maintainers weren't keen on this when I first mentioned it. |
| 32 |
|
| 33 |
Daniel |
Archives