[libvirt] [PATCH 0/6] CPU selection fixes and tests

Eric Blake eblake at redhat.com
Fri Apr 16 17:24:19 UTC 2010


On 04/16/2010 11:01 AM, Jiri Denemark wrote:
> 4 out of the 8 tests added by patch 2/6 fail with current libvirt. After
> 5/6 some of them pass and some of them fail in a different way. After
> 6/6 all of them pass.

I would suggest that before pushing, you reorder the patches so that
'make check' passes for every stage of the patch series.  This makes
patch bisection easier in the future (if we are trying to hunt down a
regression, it's better if every commit builds independently).
Admittedly, it can look a bit odd seeing the commit that fixes the bug
before the commit that introduces the test, even though they were
developed in the opposite order, but you get used to it.  And 'git
rebase -i' makes it so easy to do.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100416/d598bdf0/attachment-0001.sig>


More information about the libvir-list mailing list