On Wed, Nov 25, 2009 at 11:27:48AM +0000, Matthew Booth wrote: > On 25/11/09 11:25, Daniel P. Berrange wrote: > >On Mon, Nov 23, 2009 at 01:48:30PM +0000, Matthew Booth wrote: > >>* tests/qemuxml2argvtest.c: Test legacy syntax for QEMU argument > >>generation > >>* tests/qemuxml2argvdata/qemuxml2argv-console-legacy.(xml|args): Add test > >>data > > > >This test doesn't seem to test anything we're not already testing ? > > Where's it currently tested? I couldn't see it. I also ran a make check > with this test in place before and after the recent tty= fix. It failed > before and succeeded afterwards. > > That said, I'm not entirely sure what the console-compat test is > testing. Is it possible that the test was supposed to be in there? Since you have both <serial> and <console> elements there, <console> is ignored. Essentially <console> is just a pointer to the first serial port. <console> would only have been looked at if you hadn't had the <serial> tag there, in which case it would have been copied to the serial tag Regards, Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :| -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list