<div dir="ltr">Hi,<div>Indeed good split,  unfortunately it cause crash without any errors in log, when usbdk installed and usbclerk not.</div><div>We will fix it and will send v3 patches ASAP. </div><div>Also we fixed second scenario mentioned by Fabiano, and the first one should be fixed by patch to libusb.</div><div>Thanks, </div><div>Kirill.<br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 12, 2015 at 7:21 PM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey,<br>
<br>
While reviewing the UsbDk patches, I realized the second patch was too big for<br>
something I can easily review, and that splitting the changes adding the<br>
'use_usbclerk' variable and changing the #ifdef G_OS_WIN32 to if (use_usbclerk)<br>
would probably make the series easier to review. This is what this series attempts<br>
to do. I've only compile tested it on linux and Windows, I haven't tried to run it, so<br>
I most likely added some bugs ;) Let me know what you think about the splitting (it's<br>
not just a split, there are some differences between a branch with the initial patches<br>
applied and this one).<br>
<span><font color="#888888"><br>
Christophe<br>
<br>
</font></span></blockquote></div><br></div></div></div>