[Libvirt-cim] [PATCH] Add hypervisor version information to the Caption field of the VSMS

[Kaitlin Rupert]

> KR> If so, formatting the hypervisor version in this way might make it
> KR> difficult to parse the string.  You'd end up with something like:
> 
> KR> "QEMU 0.9.1 myotherinfohere"
> 
> I hear complaints but no suggestions :)

Fair enough.  =)  You could add some kind of demarcation between items, 
but you'd have to ensure that demarcation won't potentially show up as 
part of a string.

You could also do something like:  "Hypervisor:  QEMU 0.9.1"

Or, we can just handle that case when it happens.  I have no real 
complaints with it how it is.

> 
> We could certainly add an attribute to the service, if you'd rather.
> 
> 

Naa.. adding yet another attribute isn't worthwhile.  I agree with the 
choice you made here.

I forgot to note - I tested this out.  +1 in that regard.
-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com