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@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 :|