On 05/07/2015 11:48 AM, Laine Stump wrote: > On 05/07/2015 11:30 AM, Cole Robinson wrote: >> My commit 7b9de914 added some aarch64 CPU test cases. I wanted to test >> two different code paths but inadvertently added two of the same test >> cases. >> >> The second code path (using <cpu><model>host</model</cpu>) isn't easily >> exercised via the qemu tests anyways, I'll need to look elsewhere. >> >> Regardless, remove the redundant tests for now >> --- >> .../qemuxml2argv-aarch64-cpu-model-host.args | 5 ---- >> .../qemuxml2argv-aarch64-cpu-model-host.xml | 28 ---------------------- >> tests/qemuxml2argvtest.c | 3 --- >> .../qemuxml2argv-aarch64-cpu-passthrough.xml | 27 --------------------- >> tests/qemuxml2xmltest.c | 2 -- >> 5 files changed, 65 deletions(-) >> delete mode 100644 tests/qemuxml2argvdata/qemuxml2argv-aarch64-cpu-model-host.args >> delete mode 100644 tests/qemuxml2argvdata/qemuxml2argv-aarch64-cpu-model-host.xml >> delete mode 100644 tests/qemuxml2xmloutdata/qemuxml2argv-aarch64-cpu-passthrough.xml > > You also delete a redundant xml file in qemuxml2xmloutdata (since it's > identical to the file in qemuxml2argvdata). > > > ACK. > Thanks, pushed - Cole -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list