On 21.11.2012 18:53, Eric Blake wrote: > On 11/21/2012 10:43 AM, Michal Privoznik wrote: >> On 20.11.2012 19:47, Ján Tomko wrote: >>> Commit a4c19459aa8634c43b51e8138fb1d7eec4c17824 missed the functionality >>> for redirected USB devices (fixed in 1/4), documentation and tests. >>> >>> https://bugzilla.redhat.com/show_bug.cgi?id=805414 >>> >>> >> >> I went ahead, ACKed and pushed this. However, meanwhile I've: >> -split 2/4 patch into two smaller ones >> -split huge line in 3/4 >> -fixed space at EOL in 4/4 > > I think you should undo this fix, and match actual qemu output (even if > it DOES have a trailing space), as that particular file should be exempt > from whitespace enforcement. (If git gives you problems for pushing a > trailing space, ping me on IRC, and I'll help you figure out what to tweak). > Okay, I've fixed it. Surprisingly, there was no problem pushing the previous version. In fact, we don't run the syntax-check rule on tests/qemuhelpdata/* as I learned after digging into things. Anyway, I've proposed patch on qemu upstream list: http://lists.nongnu.org/archive/html/qemu-devel/2012-11/msg02525.html Michal -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list