From debbugs-submit-bounces@debbugs.gnu.org Tue Feb 22 03:40:40 2022 Received: (at 54069) by debbugs.gnu.org; 22 Feb 2022 08:40:40 +0000 Received: from localhost ([127.0.0.1]:39968 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1nMQiu-0001Of-9I for submit@debbugs.gnu.org; Tue, 22 Feb 2022 03:40:40 -0500 Received: from mout-p-101.mailbox.org ([80.241.56.151]:46260) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1nMQis-0001OC-8m for 54069@debbugs.gnu.org; Tue, 22 Feb 2022 03:40:39 -0500 Received: from smtp1.mailbox.org (smtp1.mailbox.org [IPv6:2001:67c:2050:105:465:1:1:0]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-101.mailbox.org (Postfix) with ESMTPS id 4K2t0Y6lrDz9sR7; Tue, 22 Feb 2022 09:40:29 +0100 (CET) Date: Tue, 22 Feb 2022 09:40:25 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=brendan.scot; s=MBO0001; t=1645519228; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=aiqs6Mw/t7SCtgjuECTujh/DOL0/9VaYRmIJUYriJXk=; b=qcx8lViZmICUGTrIVw2jVJgDHBii3YEfU9Iox1F0sQJD97HIfB1U+HKg1/ab9F43esr8Rh c+DpzgQwz/w9Ww1u8AITsQA+iLG3k+7TtvwmQFFGTVU0HtzedXxmzCZhNIRDduXAHN/Qb2 n1o1su8QSjgbaBHLwCVjClLT/s+75wMUgodWTx5qn9XsCmXYU7KUWhH92ZhBJbP3fE0Lpf uJjGKDe3NIwaIhYTJpPmoI9w/Oio9K8cOoqeBgeNS+ddS8a/2KwgiHJZlpv8oQit9o0YPT hkMEslQXSzXUxixdBrtZ2z46L80ey14waY9SK6IhYqp/mGYLTbKL6vwmcyMh6A== From: Brendan Tildesley To: Maxim Cournoyer Message-ID: <1283228187.20109.1645519225656@office.mailbox.org> In-Reply-To: <87r17wnh3d.fsf_-_@gmail.com> References: <20220220095557.1497-1-mail@brendan.scot> <20220220095557.1497-2-mail@brendan.scot> <87r17wnh3d.fsf_-_@gmail.com> Subject: Re: bug#54069: [PATCH 0/2] gnu: pciutils: Unbundle pci.ids and use latest. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Priority: 3 Importance: Normal X-Spam-Score: -0.7 (/) X-Debbugs-Envelope-To: 54069 Cc: 54069@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -1.7 (-) > On 02/21/2022 8:40 PM Maxim Cournoyer wrote: > > > Hi, > > Brendan Tildesley writes: ... > It seems to me perhaps a more proper fix would be to have hwdata used as > an input, replacing the bundled copy of the pci.ids file file, and then > letting pciutils proceed as usual? Disabling compression makes the > database about 4.3 times larger (1.2 MiB instead of 0.273 MiB). > I mistakenly thought they were not gzip'd in the first place. Should we have a separate output/package for each of the 3 files, and then embed their paths as needed? > I feel like applications parsing pci.ids themselves should probably rely > on the pciutils library API instead (libpci.so) to retrieve or filter > PCI identifiers without being concerned with the actual format of the > database. > libosinfo also pulls in its own pci.ids. I searched nix and found there are several more programs that refer to the files directly. > What do others think? > > Maxim