Patchwork gnu: libosinfo: Use stable URL for pci.ids.

login
register
mail settings
Submitter Ricardo Wurmus
Date Dec. 11, 2016, 2:49 p.m.
Message ID <20161211144922.12693-1-rekado@elephly.net>
Download mbox | patch
Permalink /patch/18349/
State New
Headers show

Comments

Ricardo Wurmus - Dec. 11, 2016, 2:49 p.m.
* gnu/packages/qemu.scm (libosinfo)[native-inputs]: Use a stable URL for
the pci.ids file.
---
 gnu/packages/qemu.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Marius Bakke - Dec. 11, 2016, 5:48 p.m.
Ricardo Wurmus <rekado@elephly.net> writes:

> * gnu/packages/qemu.scm (libosinfo)[native-inputs]: Use a stable URL for
> the pci.ids file.

The patch LGTM, but I think it would be better to upgrade to libosinfo
1.0.0 which AFAICT separates this out to a 'osinfo-db' package. WDYT?

But I don't have any objections to applying this in the meantime, if you
would rather not go down that path right now :-)

Patch

diff --git a/gnu/packages/qemu.scm b/gnu/packages/qemu.scm
index f06b52f..805bb33 100644
--- a/gnu/packages/qemu.scm
+++ b/gnu/packages/qemu.scm
@@ -230,10 +230,10 @@  server and embedded PowerPC, and S390 guests.")
        ("pci.ids"
         ,(origin
            (method url-fetch)
-           (uri "http://pciids.sourceforge.net/v2.2/pci.ids")
+           (uri "https://raw.githubusercontent.com/pciutils/pciids/f9477789526f9d380bc57aa92e357c521738d5dd/pci.ids")
            (sha256
             (base32
-             "0h8v0lrlrxkfnjiwnwiq86zyvb8qa2n3844dp1m01lh2nb2fliqw"))))
+             "0g6dbwlamagxqxvng67xng3w2x56c0np4md1v1p1jn32qw518az0"))))
        ("usb.ids"
         ,(origin
            (method url-fetch)