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(a)redhat.com>
> ---
> 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