Re: [PATCH] virsh: trim aliases from -h output

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/16/2012 09:24 PM, Osier Yang wrote:
> On 2012年03月17日 03:23, Eric Blake wrote:
>> Commit af3f9aab taught 'virsh help' to ignore command aliases,
>> but forgot 'virsh -h'.
>>
>> * tools/virsh.c (vshUsage): Handle aliases.
>> ---
>>   tools/virsh.c |    5 ++++-
>>   1 files changed, 4 insertions(+), 1 deletions(-)
>>
>> diff --git a/tools/virsh.c b/tools/virsh.c
>> index 19f9bbe..e48c56a 100644
>> --- a/tools/virsh.c
>> +++ b/tools/virsh.c
>> @@ -19721,8 +19721,11 @@ vshUsage(void)
>>           fprintf(stdout, _(" %s (help keyword '%s')\n"), grp->name,
>> grp->keyword);
>>
>>           for (cmd = grp->commands; cmd->name; cmd++)
>> +            if (cmd->flags&  VSH_CMD_FLAG_ALIAS)
>> +                continue;
>>               fprintf(stdout,
>> -                    "    %-30s %s\n", cmd->name,
>> _(vshCmddefGetInfo(cmd, "help")));
>> +                    "    %-30s %s\n", cmd->name,
>> +                    _(vshCmddefGetInfo(cmd, "help")));
>>

Oh my - I completely missed the {} (emailed the wrong version).

>>           fprintf(stdout, "\n");
>>       }
> 
> ACK

I've pushed the corrected patch.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]