Re: [libvirt] [PATCH] screenshot: Expose the new API in virsh
On 05/13/2011 08:02 AM, Michal Privoznik wrote:
* tools/virsh.c: Add screenshot command
* tools/virsh.pod: Document new command
* src/libvirt.c: Fix off-be-one error
---
src/libvirt.c | 2 +-
tools/virsh.c | 97 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
tools/virsh.pod | 9 +++++
3 files changed, 107 insertions(+), 1 deletions(-)
+
+static const vshCmdOptDef opts_screenshot[] = {
+ {"domain", VSH_OT_DATA, VSH_OFLAG_REQ, N_("domain name, id or
uuid")},
+ {"file", VSH_OT_DATA, VSH_OFLAG_REQ, N_("where to store the
screenshot")},
+ {"screen", VSH_OT_INT, VSH_OFLAG_NONE, N_("ID of a screen to take
screenshot of")},
+ {NULL, 0, 0, NULL}
+};
I still think we want the file to be optional (we can generate a decent
name if one wasn't provided).
--
Eric Blake eblake(a)redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
Attachments:
- signature.asc (application/pgp-signature — 619 bytes)