[et-mgmt-tools] [PATCH][RESEND] check of the required arguments
hbrock at redhat.com
Mon Mar 12 21:31:40 UTC 2007
Saori Fukuta wrote:
> I have no reply about following patch.
> So, I made it again for the current virt-install.
> I am looking forward to hearing your comment.
> If everything is OK, please apply it.
> On Tue, 06 Mar 2007 18:09:01 +0900 Saori Fukuta wrote:
>> So, here's what I'm going to suggest. I add two functions that check
>> the length of values, and if the value type is string I defined them
>> as a callback options .
>> They are called by following option:
>> long option (short option)
>> --name (-n)
>> --uuid (-u)
>> --file (-f)
>> --mac (-m)
>> --bridge (-b)
>> --cdrom (-c)
>> --location (-l)
> add "--keymap"
>> There is one exception. "--extra-args" doesn't call, because the option
>> has default value and that is ''.
Thanks Saori-san, patch applied. Appears to work perfectly.
Red Hat Virtualization Group http://redhat.com/virtualization
Hugh Brock | virt-manager http://virt-manager.org
hbrock at redhat.com | virtualization library http://libvirt.org
More information about the et-mgmt-tools