Re: [PATCH 2/2] git-completion.bash: prevent 'git help' from searching for git repository
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Oct 28, 2009, 06:11 UTC
- Message-ID
- <7viqe0yrnu.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <20091027133040.28836.qmail@fe20635a713ff7.315fe32.mid.smarden.org>
Gerrit Pape <pape@smarden.org> writes:
> Hi Junio, I suggest to apply this patch from Johannes to master. > > Thanks, Gerrit.
Could you help by coming up with a suitable log message?
It's a bit too much to ask me to hunt for ancient discussion to correct the <<all the ack go here>> myself to describe what the issue was, especially when I wasn't heavily involved in the review of the patch itself. My impression is that the original description of the problem and the solution in your first message does not apply to what Dscho and you agreed to be the best solution.
Show 49 quoted lines
>
> On Fri, Sep 04, 2009 at 11:09:36AM +0000, Gerrit Pape wrote:
>> On Fri, Sep 04, 2009 at 12:22:36PM +0200, Johannes Schindelin wrote:
>> > -- snipsnap --
>> > [PATCH] git help -a: do not look for a repository
>>
>> Perfect, thanks.
>>
>> Acked-by: Gerrit Pape <pape@smarden.org>
>>
>> > Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
>> >
>> > ---
>> >
>> > builtin-help.c | 6 +++---
>> > 1 files changed, 3 insertions(+), 3 deletions(-)
>> >
>> > diff --git a/builtin-help.c b/builtin-help.c
>> > index e1eba77..719aa23 100644
>> > --- a/builtin-help.c
>> > +++ b/builtin-help.c
>> > @@ -416,9 +416,6 @@ int cmd_help(int argc, const char **argv, const char *prefix)
>> > const char *alias;
>> > load_command_list("git-", &main_cmds, &other_cmds);
>> >
>> > - setup_git_directory_gently(&nongit);
>> > - git_config(git_help_config, NULL);
>> > -
>> > argc = parse_options(argc, argv, prefix, builtin_help_options,
>> > builtin_help_usage, 0);
>> >
>> > @@ -429,6 +426,9 @@ int cmd_help(int argc, const char **argv, const char *prefix)
>> > return 0;
>> > }
>> >
>> > + setup_git_directory_gently(&nongit);
>> > + git_config(git_help_config, NULL);
>> > +
>> > if (!argv[0]) {
>> > printf("usage: %s\n\n", git_usage_string);
>> > list_common_cmds_help();
>> > --
>> > To unsubscribe from this list: send the line "unsubscribe git" in
>> > the body of a message to majordomo@vger.kernel.org
>> > More majordomo info at http://vger.kernel.org/majordomo-info.html
>> --
>> To unsubscribe from this list: send the line "unsubscribe git" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html