On Tue, 2017-01-10 at 11:50 -0500, John Ferlan wrote: > > Seems reasonable - it builds for me ;-). I did take a cursory scan > through the changes - they all seem valid. > > ACK series and can push everything before the release once/if the > news > patches series on list can come to a resolution (otherwise, I'll use > news.html.in as this one does). Doubtful anyone has reservations > about > removing all the cruft, but I will let the ACK sit for the rest of > today > to see if anyone gripes ;-). > > > John > > FWIW: I had to cleanup a few make syntax-check warnings in > vbox_impl.c > due to not needing extra spaces on preprocessor indents (#define > instead > of # define): >
Whoops indeed - I've forgot to run make syntax-check before final submission. Those were the result of removing wrapping #if..#endif preprocessor directives that were used for old vbox versions which reduced nesting level. Regards, Dawid -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
