After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 581621 - Update vendor list in display-name.c
Update vendor list in display-name.c
Status: RESOLVED FIXED
Product: gnome-desktop
Classification: Core
Component: libgnome-desktop
unspecified
Other All
: Normal normal
: ---
Assigned To: Desktop Maintainers
Desktop Maintainers
: 579024 (view as bug list)
Depends on:
Blocks: randr-tracker 579024
 
 
Reported: 2009-05-06 17:37 UTC by Federico Mena Quintero
Modified: 2009-07-28 19:57 UTC
See Also:
GNOME target: ---
GNOME version: 2.27/2.28


Attachments
read the file (1.40 KB, patch)
2009-05-06 20:23 UTC, Matthias Clasen
committed Details | Review

Description Federico Mena Quintero 2009-05-06 17:37:09 UTC
Ajax tells me that the vendor list in display-name.c is kind of outdated.  There is a much more complete version here:

http://git.fedorahosted.org/git/?p=hwdata.git;a=blob_plain;f=pnp.ids;hb=HEAD

(In an ideal case, we would just share that file, but it seems to be a Red Hat-ism currently...)
Comment 1 Matthias Clasen 2009-05-06 20:23:55 UTC
Created attachment 134126 [details] [review]
read the file

Here is the patch I'm going to put into F11. 

Might make sense to do something like this and make the location of the file configurable...
Comment 2 Vincent Untz 2009-06-16 23:01:59 UTC
What about:

 + add a configure flag to specify a location of such a file;
 + hack autogen.sh to wget the file from git so that on distcheck, the file ends
   in the tarball;
 + only install the file during "make install" if the configure flag wasn't used

This way, distros not using hwdata can easily benefit from the data there. And distros using will still use it, without any change (no new file in gnome-desktop).

Alternatively, can't we make this file part of some xorg package? :-)
Comment 3 Vincent Untz 2009-06-16 23:16:03 UTC
Hrm, there are some differences in the lists. I just checked 4 lines and:

 + AIC is "AG Neovo" for gnome-desktop and "Arnos Insturments & Computer Systems" for hwdata (type in Instruments, btw)
 + DEL is "Dell" for gnome-desktop and "Deltec Corporation" for hwdata
 + ABP doesn't exist in hwdata

(the 4th line was ACR and it was the same)

So, hrm, who's right? :-)
Comment 4 Vincent Untz 2009-06-16 23:45:43 UTC
*** Bug 579024 has been marked as a duplicate of this bug. ***
Comment 5 Adam Jackson 2009-07-07 22:40:56 UTC
pnp.ids was generated from Microsoft's registry of same.  Even that contained several duplicates, so it's clearly not perfect.  I suspect the dominant use for it is going to be monitor IDs, so I'm entirely willing to call DEL Dell and not Deltec.

I could easily make a git module for it in Xorg too.  hwdata is really just the metamodule we use in fedora for pci.ids and friends, so pulling pnp.ids from somewhere else is completely par for the course.
Comment 6 Vincent Untz 2009-07-28 19:57:00 UTC
(In reply to comment #5)
> I could easily make a git module for it in Xorg too.  hwdata is really just the
> metamodule we use in fedora for pci.ids and friends, so pulling pnp.ids from
> somewhere else is completely par for the course.

Up to you, if you think it's a good idea. I've implemented what I suggested in comment #2 to make use of all this right now: there's a --with-pnp-ids-path configure option now.

I'm a bit worried that we keep the hash table in memory all the time -- it's probably around 50kB since that's the size of the file. So gnome-settings-daemon will be a bit larger. I've opened bug 590059 for this.