[libvirt] [PATCH] virsh: Two new fields for command domblklist

[Eric Blake]

On 01/11/2012 06:13 AM, Osier Yang wrote:
> Disk "type" and "device" are generally interesting stuffs the
> user who wants to known too. Though it's likely breaks the script
> which parses the command output by field (e.g. awk), putting
> them in the head is better, as it coheres with the user's habit
> more. (Trying to find the small word (type, device) after a
> very long file path (could be) sounds not a good idea).
> ---
>  tools/virsh.c   |   14 +++++++++++---
>  tools/virsh.pod |   14 +++++++-------
>  2 files changed, 18 insertions(+), 10 deletions(-)

Since this risks breaking existing scripts that were depending on the
old column ordering, I'd prefer that you resubmit this as a v2 patch,
and add a new option (perhaps domblklist --full), and only output the
new columns when the new option is in use.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120111/1baa6f93/attachment-0001.sig>