HI Anatoly, >Do all other distros have lspci in package called pciutils? If not, i >prefer to keep it the way it is.
Your original patch have: + if ret != 0: + print("'lspci' not found - please install 'lspci'") And I suggest to consider changing it to: >> + if ret != 0: >> + print("'lspci' not found - please install pciutils') > Sorry about my ignorance: which distro has a package named "lspci", if at all? The official project that include the lspci utility is called "pciutils": see: http://mj.ucw.cz/sw/pciutils/ You can see that a package named "pciutils" is available in great many distros, like: Fedora, OpenSuSE, CentOS, RHEL, Ubuntu, Debian, Mandriva and Mageia, according to the following links: http://www.rpmfind.net/linux/rpm2html/search.php?query=pciutils&submit=Search+... https://packages.ubuntu.com/trusty/pciutils https://packages.debian.org/search?keywords=pciutils https://rpms.remirepo.net/rpmphp/zoom.php?rpm=pciutils Regards, Rami Rosen