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@xxxxxxx> > > > > > > > > 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@xxxxxxxxxx> > > > > > > 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. 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. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel