[Spice-devel] [spice-gtk v2] usb-device-manager: Set UsbDk backend on libusb 1.0.22

[Victor Toso]

Hi Daniel,

Thanks for the clarification.

On Tue, Aug 07, 2018 at 11:35:00AM +0100, Daniel P. Berrangé wrote:
> On Tue, Aug 07, 2018 at 06:25:05AM -0400, Frediano Ziglio wrote:
> > > 
> > > Hi,
> > > 
> > > On Tue, Aug 07, 2018 at 06:02:41AM -0400, Frediano Ziglio wrote:
> > > > > 
> > > > > From: 王凡 <wangfan1987.good at 163.com>
> > > > > 
> > > > > In libusb 1.0.21, UsbDk is enabled at compile time with --enable-usbdk
> > > > > option. In libusb 1.0.22, the backend is selected at runtime.
> > > > > 
> > > > > This patch avoids a crash when spice client is compiled against libusb
> > > > > 1.0.22 and UsbDk is not installed.
> > > > > 
> > > > > * Enabling dynamic backend selection in Windows
> > > > > https://github.com/libusb/libusb/commit/54884e84d024e761450287ab56aca
> > > > > 
> > > > > https://gitlab.freedesktop.org/spice/spice-gtk/issues/74
> > > > > Signed-off-by: Victor Toso <victortoso at redhat.com>
> > > > 
> > > > No signed off by 王凡 (I know is the author) ?
> > > 
> > > Is it a hard requirement when the person is author of patch?
> > 
> > Not sure, that's why the question. In some cases the signed-off is used
> > for licenses issues (like saying "I, the author, allow the owner to change
> > license" or something similar).
> 
> The "Signed-off-by" tag is an attestation the person signing
> that the work complies with  https://developercertificate.org/
> 
> It it not a hard requirement for the original patch author to
> have provided a sign-off, but the implication is that in such
> cases the non-author who does apply a sign off is saying that
> they believe the original author has complied with the rules in
> the DCO.

I see now, my mistake.

The reason I put S-o-B was that I changed the patch and included
the commit log.

> So here Victor is attesting that he believes 王凡 was in
> compliance with the DCO when submitting the patch. How Victor
> has such confidence is up to him to decide - the easy way is to
> have required the original author to provide a S-o-B :-)  This
> patch, however, is trivial and is exactly what any author would
> have written for the stated problem.

Thanks,
Victor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20180807/5fc5b10f/attachment.sig>