[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 2/2] virsh: add new --details option to vol-list



On 06/19/2010 01:46 AM, Eric Blake wrote:
On 06/17/2010 07:21 PM, Justin Clift wrote:
On 06/16/2010 10:28 PM, Justin Clift wrote:
<snip>
Name                                Type   Capacity   Allocation
        Path
----------------------------------------------------------------
CentOS-5.5-x86_64-bin-DVD-1of2.iso  file   4.09 GB    4.10 GB
        /var/lib/libvirt/images/CentOS-5.5-x86_64-bin-DVD-1of2.iso
CentOS-5.5-x86_64-bin-DVD-2of2.iso  file   412.33 MB  412.74 MB
        /var/lib/libvirt/images/CentOS-5.5-x86_64-bin-DVD-2of2.iso

I'm thinking this patch is over-engineered, and should just let things
wrap.  That would then better suit scripts (important) and terminals
with wider than 80 column output.

Agreed.

Cool. There's a v2 submission later on in the day, after reworking the vol-list one:

  http://www.redhat.com/archives/libvir-list/2010-June/msg00458.html
  http://www.redhat.com/archives/libvir-list/2010-June/msg00459.html
  http://www.redhat.com/archives/libvir-list/2010-June/msg00460.html

Hope that's better. :)

Regards and best wishes,

Justin Clift

--
Salasaga  -  Open Source eLearning IDE
              http://www.salasaga.org


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]