Re: [spice-gtk PATCH v1] win-usb-dev: Set UsbDk backend on libusb 1.0.22

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Wed, Jan 9, 2019 at 3:35 PM Uri Lublin <uril@xxxxxxxxxx> wrote:
On 1/9/19 3:30 PM, Yuri Benditovich wrote:
>
>
> On Tue, Jan 8, 2019 at 3:09 PM Uri Lublin <uril@xxxxxxxxxx
> <mailto:uril@xxxxxxxxxx>> wrote:
>
>     On 1/8/19 1:35 PM, Christophe Fergeau wrote:
>      > On Tue, Jan 08, 2019 at 11:02:04AM +0200, Uri Lublin wrote:
>      >> win-usb-dev also uses libusb.
>      >> Similar to commit 3e23fff9, set libusb backend to UsbDk.
>      >> Without this patch usb devices are not seen by win-usb-dev
>      >> (windows clients), when UsbDk driver is installed.
>      >>
>      >> Signed-off-by: Uri Lublin <uril@xxxxxxxxxx <mailto:uril@xxxxxxxxxx>>
>      >> ---
>      >>
>      >> Additionally I think the code need to check that usbdk driver
>      >> is indeed installed before setting the backend.
>      >
>      > Isn't this
>      > https://gitlab.freedesktop.org/spice/spice-gtk/commit/8c0fdcbc ?
>
>     Yes, it is, sorry.
>
>     As mentioned above, I think we should do it only if usbdk driver
>     is indeed installed.
>
>
> What shall we do if usbdk is not installed?

I have a patch that checks if usbdk is installed.
I'll send it later.

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?

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]