"Jean-Noël AVILA" <jn.avila@xxxxxxx> writes:
> My rational was to be sure to put the environment variable out of the
> way once the script has been sourced. I can make two alternative
> definitions of __git_list_all_commands () depending on the presence of
> $AUTHORIZED_CMD_LIST if you are worried about performance.
Actually, it does not matter, as once __git_list_all_commands is
run, its result is kept in a variable.
So Peff's
if test -z "$FAKE_COMMAND_LIST"; then
__git_cmdlist() {
git help -a | egrep '^ [a-zA-Z0-9]'
}
else
__git_cmdlist() {
printf '%s' "$FAKE_COMMAND_LIST"
}
fi
would be just as simple if not simpler, does the same thing, and is
sufficient, I think.
The t9902 test is only interested in making sure that the completion
works, and we do not want "git help -a" that omits a subcommand from
its output that is not built in your particular environment to get
in the way, which will not be an issue with this approach.
--
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