<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 9, 2019 at 3:35 PM Uri Lublin <<a href="mailto:uril@redhat.com">uril@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 1/9/19 3:30 PM, Yuri Benditovich wrote:<br>
> <br>
> <br>
> On Tue, Jan 8, 2019 at 3:09 PM Uri Lublin <<a href="mailto:uril@redhat.com" target="_blank">uril@redhat.com</a> <br>
> <mailto:<a href="mailto:uril@redhat.com" target="_blank">uril@redhat.com</a>>> wrote:<br>
> <br>
>     On 1/8/19 1:35 PM, Christophe Fergeau wrote:<br>
>      > On Tue, Jan 08, 2019 at 11:02:04AM +0200, Uri Lublin wrote:<br>
>      >> win-usb-dev also uses libusb.<br>
>      >> Similar to commit 3e23fff9, set libusb backend to UsbDk.<br>
>      >> Without this patch usb devices are not seen by win-usb-dev<br>
>      >> (windows clients), when UsbDk driver is installed.<br>
>      >><br>
>      >> Signed-off-by: Uri Lublin <<a href="mailto:uril@redhat.com" target="_blank">uril@redhat.com</a> <mailto:<a href="mailto:uril@redhat.com" target="_blank">uril@redhat.com</a>>><br>
>      >> ---<br>
>      >><br>
>      >> Additionally I think the code need to check that usbdk driver<br>
>      >> is indeed installed before setting the backend.<br>
>      ><br>
>      > Isn't this<br>
>      > <a href="https://gitlab.freedesktop.org/spice/spice-gtk/commit/8c0fdcbc" rel="noreferrer" target="_blank">https://gitlab.freedesktop.org/spice/spice-gtk/commit/8c0fdcbc</a> ?<br>
> <br>
>     Yes, it is, sorry.<br>
> <br>
>     As mentioned above, I think we should do it only if usbdk driver<br>
>     is indeed installed.<br>
> <br>
> <br>
> What shall we do if usbdk is not installed?<br>
<br>
I have a patch that checks if usbdk is installed.<br>
I'll send it later.<br></blockquote><div><br></div><div>The question is different: let's say the usbdk is not installed. What we do? Work with existing backend? If so, this should be covered in testing and we should  declare that spice-gtk supports also other backends. Correct?</div><div><br></div></div></div>