On 04/23/2013 06:40 AM, Cole Robinson wrote:
On 04/22/2013 01:45 PM, Guannan Ren wrote:
def get_net_by_name(self, name):
for net in self.nets.values():
if net.get_name() == name:
diff --git a/virtinst/VirtualHostDevice.py b/virtinst/VirtualHostDevice.py
index f5740d0..ee38242 100644
--- a/virtinst/VirtualHostDevice.py
+++ b/virtinst/VirtualHostDevice.py
@@ -215,9 +215,8 @@ class VirtualHostDeviceUSB(VirtualHostDevice):
self.vendor = nodedev.vendor_id
self.product = nodedev.product_id
- if not (self.vendor or self.product):
- self.bus = nodedev.bus
- self.device = nodedev.device
+ self.bus = nodedev.bus
+ self.device = nodedev.device
def _get_source_xml(self):
xml = ""
I know this was involved in breaking the test suite in the past, though I
think it required another virtinst change. This definitely doesn't cause
python setup.py test regressions right?
Thanks,
Cole
Sorry about this, the regression has been fixed in v2.
But there are some testing failures on my machine which are not
related to my patches,
I will try to figure them out soon later.
Guannan
_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list
