On 10-Nov-18 11:03 AM, Rami Rosen wrote:
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
Hi,
Thanks, that's good to know, will fix in v2.
--
Thanks,
Anatoly
