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 <jfer...@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 :| -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
