On 08-Nov-18 8:38 PM, Rami Rosen wrote:
Hi, Anatoly,

This is really minor nitpick, but in case you decide to send V2, I believe it should say
 >> + if ret != 0:
 >> +            print("'lspci' not found - please install pciutils')

as the name of the package containing lspci is pciutils in centos.

Do all other distros have lspci in package called pciutils? If not, i prefer to keep it the way it is.


Regards,
Rami Rosen




--
Thanks,
Anatoly

Reply via email to