]> mj.ucw.cz Git - pciutils.git/commitdiff
update-pciids: Report itself as an user agent, version included
authorMartin Mares <mj@ucw.cz>
Sun, 23 Jul 2023 13:44:03 +0000 (15:44 +0200)
committerMartin Mares <mj@ucw.cz>
Sun, 23 Jul 2023 13:44:03 +0000 (15:44 +0200)
Unfortunately, this leads to the User-Agent not containing version
of curl/wget/lynx we used.

Makefile
update-pciids.sh

index 0e878a6956d7181a51d0fd4906dbf5badaa71b58..228cb5693aaf7fa0297e295ec3539708eb1e67dc 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -103,7 +103,7 @@ lspci$(EXEEXT): LDLIBS+=$(LIBKMOD_LIBS)
 ls-kernel.o: CFLAGS+=$(LIBKMOD_CFLAGS)
 
 update-pciids: update-pciids.sh
-       sed <$< >$@ "s@^DEST=.*@DEST=$(if $(IDSDIR),$(IDSDIR)/,)$(PCI_IDS)@;s@^PCI_COMPRESSED_IDS=.*@PCI_COMPRESSED_IDS=$(PCI_COMPRESSED_IDS)@"
+       sed <$< >$@ "s@^DEST=.*@DEST=$(if $(IDSDIR),$(IDSDIR)/,)$(PCI_IDS)@;s@^PCI_COMPRESSED_IDS=.*@PCI_COMPRESSED_IDS=$(PCI_COMPRESSED_IDS)@;s@VERSION=.*@VERSION=$(VERSION)@"
        chmod +x $@
 
 # The example of use of libpci
index 1fa604e1bf782cf7bc96bbbecdd1a6f13730f158..607502ad45a51a033cc336eac5ca538e2af91202 100755 (executable)
@@ -6,6 +6,9 @@ SRC="https://pci-ids.ucw.cz/v2.2/pci.ids"
 DEST=pci.ids
 PCI_COMPRESSED_IDS=
 GREP=grep
+VERSION=unknown
+USER_AGENT=update-pciids/$VERSION
+QUIET=
 
 [ "$1" = "-q" ] && quiet=true || quiet=false
 
@@ -30,19 +33,28 @@ else
 fi
 
 if command -v curl >/dev/null 2>&1 ; then
-       DL="curl -o $DEST.new $SRC"
-       ${quiet} && DL="$DL -s -S"
+       ${quiet} && QUIET="-s -S"
+       dl ()
+       {
+               curl -o $DEST.new --user-agent "$USER_AGENT curl" $QUIET $SRC
+       }
 elif command -v wget >/dev/null 2>&1 ; then
-       DL="wget --no-timestamping -O $DEST.new $SRC"
-       ${quiet} && DL="$DL -q"
+       ${quiet} && QUIET="-q"
+       dl ()
+       {
+               wget --no-timestamping -O $DEST.new --user-agent "$USER_AGENT wget" $QUIET $SRC
+       }
 elif command -v lynx >/dev/null 2>&1 ; then
-       DL="eval lynx -source $SRC >$DEST.new"
+       dl ()
+       {
+               lynx -source -useragent="$USER_AGENT lynx" $SRC >$DEST.new
+       }
 else
        echo >&2 "update-pciids: cannot find curl, wget or lynx"
        exit 1
 fi
 
-if ! $DL ; then
+if ! dl ; then
        echo >&2 "update-pciids: download failed"
        rm -f $DEST.new
        exit 1