On Mon, Aug 28, 2017 at 12:23:19PM +0200, Pavel Hrdina wrote: > On Sun, Aug 27, 2017 at 11:04:39AM -0400, Cole Robinson wrote: > > Will be needed for future patches to pull the default video type > > setting out of XML parsing routines. > > > > Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx> > > --- > > src/conf/domain_conf.c | 1 + > > src/conf/domain_conf.h | 1 + > > src/qemu/qemu_command.c | 3 +++ > > src/qemu/qemu_domain_address.c | 1 + > > tests/domaincapsschemadata/full.xml | 1 + > > 5 files changed, 7 insertions(+) > > [...] > > > diff --git a/tests/domaincapsschemadata/full.xml b/tests/domaincapsschemadata/full.xml > > index 82a92322e..ab6ef9f2e 100644 > > --- a/tests/domaincapsschemadata/full.xml > > +++ b/tests/domaincapsschemadata/full.xml > > @@ -62,6 +62,7 @@ > > </graphics> > > <video supported='yes'> > > <enum name='modelType'> > > + <value>default</value> > > <value>vga</value> > > <value>cirrus</value> > > <value>vmvga</value> > > This hunk should not be required, but I've checked the code that fills > in domain capabilities and we claim that every video model is supported. > Well that's wrong and should be fixed. Sigh :) Ignore this comment, the code works correctly, this is a test where we format all capabilities. Pavel
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
