From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:56992) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T5tYH-0001iT-7Y for qemu-devel@nongnu.org; Mon, 27 Aug 2012 03:16:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1T5tYG-0005EN-63 for qemu-devel@nongnu.org; Mon, 27 Aug 2012 03:16:01 -0400 Received: from mail-wi0-f181.google.com ([209.85.212.181]:49794) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T5tYG-0005EC-03 for qemu-devel@nongnu.org; Mon, 27 Aug 2012 03:16:00 -0400 Received: by wibhm2 with SMTP id hm2so2122879wib.10 for ; Mon, 27 Aug 2012 00:15:58 -0700 (PDT) From: "=?UTF-8?q?Beno=C3=AEt=20Canet?=" Date: Mon, 27 Aug 2012 09:15:50 +0200 Message-Id: <1346051752-13551-1-git-send-email-benoit@irqsave.net> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PATCH V6 0/2] Add JSON output to qemu-img info List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, stefanha@linux.vnet.ibm.com, blauwirbel@gmail.com, pbonzini@redhat.com, eblake@redhat.com, xiawenc@linux.vnet.ibm.com, =?UTF-8?q?Beno=C3=AEt=20Canet?= This patchset add a JSON output mode to the qemu-img info command. It's a rewrite from scratch of the original patchset by Wenchao Xia following Anthony Liguori advices on JSON formating. the --output=(json|human) option is now mandatory on the command line. Benoît Canet (3): qapi: Add SnapshotInfo. qapi: Add ImageInfo. qemu-img: Add json output option to the info command. in v2: eblake: make some field optionals squash the two qapi patchs together fix a typo on vm_clock_nsec bcanet: fix a potential memory leak in v3: lcapitulino: remove unneeded test put '\n' at the end of json in printf statement drop the uneeded head pointer in collect_snapshots in v4: Wenchao Xia && Kevin Wolf: -Refactor to separate rate ImageInfo collection from human printing. Kevin Wolf: -Use --output=(json|human). -make the two choice exclusive and print a message if none is specified. -cosmetic '=' alignement in collect snapshots. Benoît Canet: -add full-backing-filename to the ImageInfo structure (needed for human printing) -make ImageInfo->actual_size optional depending on the context. in v5: Eric Blake: -use a constant for getopt parsing to avoid future short options collision. -make the command default to --output=human. -fix spurious whitespace change. -split vm-clock-nsec in two fields vm-clock-sec and vm-clock-nsec. -declare JSON structure as "Since 1.3" in v6: Blue Swirl: -Add missing const in getopt structure declaration. Eric Blake: -Remove spurious undef. -Use an enum instead of two boolean. Benoît Canet (2): qapi: Add SnapshotInfo and ImageInfo. qemu-img: Add json output option to the info command. Makefile | 3 +- qapi-schema.json | 64 ++++++++++++++ qemu-img.c | 259 +++++++++++++++++++++++++++++++++++++++++++++--------- 3 files changed, 282 insertions(+), 44 deletions(-) -- 1.7.9.5