Re: [libvirt] [PATCH] libvirt-tck: add junit output

On Mon, Oct 10, 2011 at 04:15:47PM -0600, Eric Blake wrote: > On 10/10/2011 03:46 PM, Guido Günther wrote: > >Hi, > >having JUnit like XML allows for easy integration into CI tools like > >Jenkins. Attach patch adds this using TAP::Formatter::JUnit. > >Does this look reasonable? > > Do we need to modify a spec file to pull in the > TAP::Formatter::JUnit module when installing the tck on Fedora? > Other than that, it seems reasonable to me, but I'd also like Dan's > feedback. Normally you don't need to add Perl modules to the SPEC because RPM will autodetect them, but because of the way the TAP system works, we *do* need to add any TAP::Formatter::* modules manually. So ACK if you add this as a Requires and BuildRequires in the RPM perl-Sys-Virt-TCK.spec.PL