Re: [libvirt] [PATCH] virsh: don't shorten incorrect arguments

2013-04-24 Thread Eric Blake
On 04/24/2013 05:28 AM, Ján Tomko wrote: > On 04/23/2013 06:05 PM, Eric Blake wrote: >> On 04/23/2013 08:08 AM, Ján Tomko wrote: >>> https://bugzilla.redhat.com/show_bug.cgi?id=949373 >>> >>> Print the whole incorrect argument as specified by the user >>> instead of the short option. >>> >> >>>

Re: [libvirt] [PATCH] virsh: don't shorten incorrect arguments

2013-04-24 Thread Ján Tomko
On 04/23/2013 06:05 PM, Eric Blake wrote: > On 04/23/2013 08:08 AM, Ján Tomko wrote: >> https://bugzilla.redhat.com/show_bug.cgi?id=949373 >> >> Print the whole incorrect argument as specified by the user >> instead of the short option. >> > >> case 'd': >> if (virStrToLong_i

Re: [libvirt] [PATCH] virsh: don't shorten incorrect arguments

2013-04-23 Thread Eric Blake
On 04/23/2013 08:08 AM, Ján Tomko wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=949373 > > Print the whole incorrect argument as specified by the user > instead of the short option. > > case 'd': > if (virStrToLong_i(optarg, NULL, 10, &debug) < 0) { > -

Re: [libvirt] [PATCH] virsh: don't shorten incorrect arguments

2013-04-23 Thread Ján Tomko
On 04/23/2013 04:25 PM, Michal Privoznik wrote: > On 23.04.2013 16:08, Ján Tomko wrote: >> https://bugzilla.redhat.com/show_bug.cgi?id=949373 >> >> Print the whole incorrect argument as specified by the user >> instead of the short option. >> >> Before: >> $ virsh --debu >> error: option '-d' requi

Re: [libvirt] [PATCH] virsh: don't shorten incorrect arguments

2013-04-23 Thread Michal Privoznik
On 23.04.2013 16:08, Ján Tomko wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=949373 > > Print the whole incorrect argument as specified by the user > instead of the short option. > > Before: > $ virsh --debu > error: option '-d' requires an argument > $ virsh ---debu > error: unsupported o

[libvirt] [PATCH] virsh: don't shorten incorrect arguments

2013-04-23 Thread Ján Tomko
https://bugzilla.redhat.com/show_bug.cgi?id=949373 Print the whole incorrect argument as specified by the user instead of the short option. Before: $ virsh --debu error: option '-d' requires an argument $ virsh ---debu error: unsupported option '- $ virsh --debu c error: option -d takes a numeric