From: Christian Couder <chriscool@tuxfamily.org>
To: Miklos Vajna <vmiklos@frugalware.org>
Cc: Johannes Schindelin <Johannes.Schindelin@gmx.de>,
Kevin Ballard <kevin@sb.org>,
Git Mailing List <git@vger.kernel.org>
Subject: Re: git help broken
Date: Fri, 1 Aug 2008 06:28:52 +0200 [thread overview]
Message-ID: <200808010628.52875.chriscool@tuxfamily.org> (raw)
In-Reply-To: <20080731000454.GJ32057@genesis.frugalware.org>
Le jeudi 31 juillet 2008, Miklos Vajna a écrit :
> On Thu, Jul 31, 2008 at 01:44:36AM +0200, Johannes Schindelin
<Johannes.Schindelin@gmx.de> wrote:
> > > And from the patch, it is pretty obvious that it does not come close
> > > to the "man" code path.
> >
> > Oh, so it was involved?
>
> Yes. The command list is no longer loaded automatically and the default
> for non-git commands on git help foo was 'gitfoo', I guess for
> gittutorial and such manpages.
Yeah, and it seems that "git help help" doesn't work any more since my patch
that does that:
2dce956e397be8a769624e8693565b8a30056060 help: check early if we have a
command, if not try a documentation topic
I didn't realize that the function "is_git_command" would not work
for "help".
Perhaps there should be another function like is_git_command that could work
using the "struct cmd_struct commands" defined in "handle_internal_command"
in "git.c". That may be faster and more accurate.
Unfortunately I won't have time to work on this as I will leave for vacation
in a few hours. Sorry to leave this behind.
Best regards,
Christian.
next prev parent reply other threads:[~2008-08-01 4:26 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-30 20:52 git help broken Kevin Ballard
2008-07-30 22:38 ` [PATCH] builtin-help: always load_command_list() in cmd_help() Miklos Vajna
2008-07-30 22:50 ` Kevin Ballard
2008-07-30 23:42 ` git help broken Johannes Schindelin
2008-07-30 23:44 ` Johannes Schindelin
2008-07-31 0:04 ` Miklos Vajna
2008-08-01 4:28 ` Christian Couder [this message]
2008-08-01 14:41 ` Miklos Vajna
2008-08-02 8:08 ` [PATCH] Builtin git-help Miklos Vajna
2008-08-21 2:51 ` git help broken Christian Couder
2008-08-21 14:21 ` [PATCH] Fix 'git help help' Miklos Vajna
2008-08-22 4:40 ` Christian Couder
2008-07-30 23:45 ` git help broken Kevin Ballard
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200808010628.52875.chriscool@tuxfamily.org \
--to=chriscool@tuxfamily.org \
--cc=Johannes.Schindelin@gmx.de \
--cc=git@vger.kernel.org \
--cc=kevin@sb.org \
--cc=vmiklos@frugalware.org \
/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.
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 NNTP newsgroup(s).