On Mon, May 18, 2009 at 01:57:48PM -0400, Cole Robinson wrote:
Let the underlying driver tell us what it supports. At the moment we are at least erroneously denying floppy eject/insert.
Last time I posted this, it was pointed out that 'type' now needs to be properly escaped for use in the xml: while true, that's a problem in numerous other places in virsh, so I've skipped it with this patch. --- src/virsh.c | 7 ------- 1 files changed, 0 insertions(+), 7 deletions(-)
diff --git a/src/virsh.c b/src/virsh.c index ce90d3d..865b6e5 100644 --- a/src/virsh.c +++ b/src/virsh.c @@ -5509,13 +5509,6 @@ cmdAttachDisk(vshControl *ctl, const vshCmd *cmd) type = vshCommandOptString(cmd, "type", NULL); mode = vshCommandOptString(cmd, "mode", NULL);
- if (type) { - if (STRNEQ(type, "cdrom") && STRNEQ(type, "disk")) { - vshError(ctl, FALSE, _("No support %s in command 'attach-disk'"), type); - goto cleanup; - } - } - if (driver) { if (STREQ(driver, "file") || STREQ(driver, "tap")) { isFile = 1; --
ACK. Could have sworn we decided to get rid of this months ago. Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|