Re: [PATCH 3/5] generate-cmdlist: parse common group commands

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

 



On Thu, May 14, 2015 at 4:58 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Sébastien Guimmara  <sebastien.guimmara@xxxxxxxxx> writes:
>> From: Eric Sunshine <sunshine@xxxxxxxxxxxxxx>
>>
>> Parse the [common] block to create the array of group descriptions:
>>
>> static char *common_cmd_groups[] = {
>>     N_("starting a working area"),
>>     N_("working on the current change"),
>>     N_("working with others"),
>>     N_("examining the history and state"),
>>     N_("growing, marking and tweaking your history"),
>> };
>>
>> then map each element of common_cmds[] to a group via its index:
>>
>> static struct cmdname_help common_cmds[] = {
>>     {"add", N_("Add file contents to the index"), 1},
>>     {"branch", N_("List, create, or delete branches"), 4},
>>     {"checkout", N_("Checkout a branch or paths to the ..."), 4},
>>     {"clone", N_("Clone a repository into a new directory"), 0},
>>     {"commit", N_("Record changes to the repository"), 4},
>>     ...
>> };
>>
>> so that 'git help' can print those commands grouped by theme.
>>
>> Only commands tagged with an attribute from [common] are emitted to
>> common_cmds[].
>>
>> [commit message by Sébastien Guimmara <sebastien.guimmara@xxxxxxxxx>]
>>
>> Signed-off-by: Eric Sunshine <sunshine@xxxxxxxxxxxxxx>
>> Signed-off-by: Sébastien Guimmara <sebastien.guimmara@xxxxxxxxx>
>> ---
>
> I seem to be getting an empty common_cmds[] list after this step
> (with GNU Awk 4.0.1, ICIM).

Indeed. I haven't had a chance to look at this version of the series
yet, but a quick glance shows that this is because patch 2/5 uses
"common-" as a prefix rather than as a standalone tag. That is, lines
in patch 2/5 like this:

    git-add  mainporcelain  common-worktree

should be:

    git-add  mainporcelain  common worktree

as proposed here[1]. And, then patch 4/5 should drop the standalone
"common" tag rather than the "common-" prefix.

[1]: http://article.gmane.org/gmane.comp.version-control.git/268756
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]