[et-mgmt-tools] [PATCH][RESEND] check of the required arguments
Hugh Brock
hbrock at redhat.com
Mon Mar 12 21:31:40 UTC 2007
Saori Fukuta wrote:
> Hi,
>
> 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)
>> --connect
>> --cdrom (-c)
>> --os-type
>> --os-variant
>> --arch
>> --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.
--Hugh
--
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
mailing list