From: "Jean-Noël AVILA" <avila.jn@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org, "Torsten Bögershausen" <tboegi@web.de>,
"Jonathan Nieder" <jrnieder@gmail.com>,
"Jeff King" <peff@peff.net>,
"Nguyễn Thái Ngọc Duy" <pclouds@gmail.com>,
"Felipe Contreras" <felipe.contreras@gmail.com>
Subject: Re: [PATCH 2/2] help --standard: list standard commands
Date: Sat, 19 Jan 2013 11:32:51 +0100 [thread overview]
Message-ID: <201301191132.51808.avila.jn@gmail.com> (raw)
In-Reply-To: <7vk3r9ob22.fsf_-_@alter.siamese.dyndns.org>
Le samedi 19 janvier 2013 09:02:13, Junio C Hamano a écrit :
> If you run "make" on a branch that adds "git check-ignore", checkout
> an older branch that did not know about the command without "make clean",
> and the run t9902 test, the completion script fails to exclude the
> "check-ignore" command from candidates to complete "check".
>
> This is because the completion script asks "git help -a" to show
> every executable that begins with "git-" in the GIT_EXEC_PATH, and
> because we run tests with GIT_EXEC_PATH set to the top of the
> working tree, that has the executable we just built, in order to
> test these before installing. Leftover "git check-ignore" that we
> did not build for the current version gets in the way.
>
> One way to solve this is to restrict the completion to only the
> commands we know about.
>
> Note that this will make the completion useless in real life for
> some people, as they do want to get their custom commands on their
> $PATH to be included in the completion.
>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
> ---
>
> * This is not a real patch, for the reasons stated.
>
> builtin/help.c | 17 ++++++++++++++---
> command-list.txt | 4 ++--
> contrib/completion/git-completion.bash | 14 +-------------
> generate-cmdlist.sh | 13 ++++++++++++-
> help.c | 30 ++++++++++++++++++++++++++++--
> help.h | 1 +
> 6 files changed, 58 insertions(+), 21 deletions(-)
>
> diff --git a/builtin/help.c b/builtin/help.c
> index 6067a61..32e7d64 100644
> --- a/builtin/help.c
> +++ b/builtin/help.c
> @@ -35,11 +35,16 @@ enum help_format {
>
> static const char *html_path;
>
> -static int show_all = 0;
> +#define HELP_SHOW_ALL 1
> +#define HELP_SHOW_STANDARD 2
> +static int show_what;
> static unsigned int colopts;
> static enum help_format help_format = HELP_FORMAT_NONE;
> static struct option builtin_help_options[] = {
> - OPT_BOOLEAN('a', "all", &show_all, N_("print all available commands")),
> + OPT_SET_INT('a', "all", &show_what, N_("print all available commands"),
> + HELP_SHOW_ALL),
> + OPT_SET_INT(0, "standard", &show_what, N_("print all available
> commands"), + HELP_SHOW_STANDARD),
> OPT_SET_INT('m', "man", &help_format, N_("show man page"),
> HELP_FORMAT_MAN), OPT_SET_INT('w', "web", &help_format, N_("show manual in
> web browser"), HELP_FORMAT_WEB),
> @@ -424,12 +429,18 @@ int cmd_help(int argc, const char **argv, const char
> *prefix) builtin_help_usage, 0);
> parsed_help_format = help_format;
>
> - if (show_all) {
> + if (show_what == HELP_SHOW_ALL) {
> git_config(git_help_config, NULL);
> printf(_("usage: %s%s"), _(git_usage_string), "\n\n");
> list_commands(colopts, &main_cmds, &other_cmds);
> printf("%s\n", _(git_more_info_string));
> return 0;
> + } else if (show_what == HELP_SHOW_STANDARD) {
> + int i;
> + limit_to_standard(&main_cmds);
> + for (i = 0; i < main_cmds.cnt; i++)
> + printf("%s\n", main_cmds.names[i]->name);
> + return 0;
> }
>
> if (!argv[0]) {
> diff --git a/command-list.txt b/command-list.txt
> index 7e8cfec..94ce8ec 100644
> --- a/command-list.txt
> +++ b/command-list.txt
> @@ -116,8 +116,8 @@ git-show mainporcelain
> common git-show-branch ancillaryinterrogators
> git-show-index plumbinginterrogators
> git-show-ref plumbinginterrogators
> -git-sh-i18n purehelpers
> -git-sh-setup purehelpers
> +git-sh-i18n purehelpers nocomplete
> +git-sh-setup purehelpers nocomplete
> git-stash mainporcelain
> git-status mainporcelain common
> git-stripspace purehelpers
> diff --git a/contrib/completion/git-completion.bash
> b/contrib/completion/git-completion.bash index a4c48e1..46f22af 100644
> --- a/contrib/completion/git-completion.bash
> +++ b/contrib/completion/git-completion.bash
> @@ -531,23 +531,11 @@ __git_complete_strategy ()
> return 1
> }
>
> -__git_list_all_commands ()
> -{
> - local i IFS=" "$'\n'
> - for i in $(git help -a|egrep '^ [a-zA-Z0-9]')
> - do
> - case $i in
> - *--*) : helper pattern;;
> - *) echo $i;;
> - esac
> - done
> -}
> -
> __git_all_commands=
> __git_compute_all_commands ()
> {
> test -n "$__git_all_commands" ||
> - __git_all_commands=$(__git_list_all_commands)
> + __git_all_commands=$(git help --standard)
> }
>
> __git_list_porcelain_commands ()
> diff --git a/generate-cmdlist.sh b/generate-cmdlist.sh
> index 9a4c9b9..7800af3 100755
> --- a/generate-cmdlist.sh
> +++ b/generate-cmdlist.sh
> @@ -9,7 +9,7 @@ struct cmdname_help {
> static struct cmdname_help common_cmds[] = {"
>
> sed -n -e 's/^git-\([^ ]*\)[ ].* common.*/\1/p' command-list.txt |
> -sort |
> +LC_ALL=C LANG=C sort |
> while read cmd
> do
> sed -n '
> @@ -20,4 +20,15 @@ do
> p
> }' "Documentation/git-$cmd.txt"
> done
> +echo "};
> +
> +static const char *standard_cmd[] = {"
> +
> +LC_ALL=C LANG=C sort command-list.txt |
> +sed -n -e '
> + /^git-[^ ]*[ ].* deprecated.*/d
> + /^git-[^ ]*[ ].* nocomplete.*/d
> + s/^git-\([^ ]*\)[ ].*/ "\1",/p
> +'
> +
> echo "};"
> diff --git a/help.c b/help.c
> index 1dfa0b0..2ad10db 100644
> --- a/help.c
> +++ b/help.c
> @@ -182,7 +182,7 @@ void load_command_list(const char *prefix,
> uniq(main_cmds);
> }
>
> - if (env_path) {
> + if (env_path && other_cmds) {
> char *paths, *path, *colon;
> path = paths = xstrdup(env_path);
> while (1) {
> @@ -201,7 +201,33 @@ void load_command_list(const char *prefix,
> sizeof(*other_cmds->names), cmdname_compare);
> uniq(other_cmds);
> }
> - exclude_cmds(other_cmds, main_cmds);
> +
> + if (other_cmds)
> + exclude_cmds(other_cmds, main_cmds);
> +}
> +
> +void limit_to_standard(struct cmdnames *cmds)
> +{
> + int src = 0, dst = 0, ref = 0;
> +
> + while (src < cmds->cnt && ref < ARRAY_SIZE(standard_cmd)) {
> + int cmp = strcmp(cmds->names[src]->name, standard_cmd[ref]);
> + if (cmp < 0) {
> + src++; /* not a standard command */
> + } else if (!cmp) {
> + if (dst != src) {
> + free(cmds->names[dst]);
> + cmds->names[dst] = cmds->names[src];
> + }
> + ref++;
> + dst++;
> + } else {
> + ref++; /* uninstalled standard command */
> + }
> + }
> + for (src = dst; src < cmds->cnt; src++)
> + free(cmds->names[src]);
> + cmds->cnt = dst;
> }
>
> void list_commands(unsigned int colopts,
> diff --git a/help.h b/help.h
> index 0ae5a12..ce0d2a5 100644
> --- a/help.h
> +++ b/help.h
> @@ -21,6 +21,7 @@ extern const char *help_unknown_cmd(const char *cmd);
> extern void load_command_list(const char *prefix,
> struct cmdnames *main_cmds,
> struct cmdnames *other_cmds);
> +extern void limit_to_standard(struct cmdnames *);
> extern void add_cmdname(struct cmdnames *cmds, const char *name, int len);
> /* Here we require that excludes is a sorted list. */
> extern void exclude_cmds(struct cmdnames *cmds, struct cmdnames
> *excludes);
With these two patches, the test passes.
next prev parent reply other threads:[~2013-01-19 10:33 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-15 13:35 [PATCH] attr: fix off-by-one directory component length calculation Nguyễn Thái Ngọc Duy
2013-01-15 16:17 ` Junio C Hamano
2013-01-15 19:14 ` Jean-Noël AVILA
2013-01-15 19:29 ` Junio C Hamano
2013-01-15 19:53 ` Jean-Noël AVILA
2013-01-15 20:49 ` Junio C Hamano
2013-01-15 23:24 ` Jeff King
2013-01-16 6:15 ` t9902 fails (Was: [PATCH] attr: fix off-by-one directory component length calculation) Torsten Bögershausen
2013-01-17 22:47 ` Jean-Noël AVILA
2013-01-18 0:04 ` Jonathan Nieder
2013-01-18 0:12 ` t9902 fails Junio C Hamano
2013-01-18 5:20 ` t9902 fails (Was: [PATCH] attr: fix off-by-one directory component length calculation) Torsten Bögershausen
2013-01-18 16:49 ` t9902 fails Junio C Hamano
2013-01-18 20:15 ` Junio C Hamano
2013-01-18 22:23 ` Jean-Noël AVILA
2013-01-18 22:37 ` Junio C Hamano
2013-01-18 22:57 ` Junio C Hamano
2013-01-19 5:38 ` Torsten Bögershausen
2013-01-19 7:52 ` Re* " Junio C Hamano
2013-01-19 8:00 ` [PATCH 1/2] help: include <common-cmds.h> only in one file Junio C Hamano
2013-01-19 8:02 ` [PATCH 2/2] help --standard: list standard commands Junio C Hamano
2013-01-19 10:32 ` Jean-Noël AVILA [this message]
2013-01-19 13:43 ` Re* t9902 fails Jean-Noël AVILA
2013-01-19 10:23 ` Jean-Noël AVILA
2013-01-16 1:08 ` [PATCH] attr: fix off-by-one directory component length calculation Duy Nguyen
2013-01-16 2:09 ` Duy Nguyen
2013-01-16 2:33 ` Junio C Hamano
2013-01-16 3:12 ` Duy Nguyen
2013-01-16 5:35 ` Junio C Hamano
2013-01-16 6:02 ` Duy Nguyen
2013-01-16 19:07 ` Junio C Hamano
2013-01-16 1:03 ` Duy Nguyen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201301191132.51808.avila.jn@gmail.com \
--to=avila.jn@gmail.com \
--cc=felipe.contreras@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jrnieder@gmail.com \
--cc=pclouds@gmail.com \
--cc=peff@peff.net \
--cc=tboegi@web.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).