Re: [libvirt] [PATCH] conf: Fix printing of 'type' and 'tty_compat' for Chr devices
On Wed, Aug 02, 2017 at 03:51:34PM -0400, John Ferlan wrote:
Commit id '0c1d8632' caused a regression in the virt-manager
test suite when formatting the <smartcard mode='passthrough'
type='spicevmc'/>.
Adust the code to print the type in it's own new helper called
virDomainChrTypeFormat and have the virDomainChrSourceDefFormat
manage just formatting the source and change to a void type since
only 0 could be returned. Adjust the callers to handle properly.
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
Although technically a CI build breaker since virt-manager test is
failing, I figured I'd let this one go through the formal review just
in case someone has agita over new function name or would like to see
things done in a different manner.
src/conf/domain_conf.c | 39 ++++++++++++++++++++++++---------------
1 file changed, 24 insertions(+), 15 deletions(-)
Can you add a test case for that, since the lack of tests is why
we have this regression in tree, and I'd feel comfortable that it
actually fixes the problem if tests show it.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|