On Fri, Sep 20, 2019 at 2:34 PM Fabiano Fidêncio <fidencio@xxxxxxxxxx> wrote: > > [snip] > > > > > > In general, it looks good and works as expected. > > > I will add my "Reviewed-by: " after we discuss the points raised. > > > > > > Another thing, please, let's sync to have the libvirt-jenkins-ci work > > > done and merged before this one gets merged. > > > > Works for me, thanks for review, I'll fix the dist script to use a shell > > script. > > > > Reviewed-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx> One last things here (thanks, Pavel, for pointing this out), I'd prefer the 'syntax' suite being called 'syntax-check' just because people are already used to the 'syntax-check' name. So, please, would you mind changing it as well? Best Regards, -- Fabiano Fidêncio -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
