All of lore.kernel.org
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Pascal Obry <pascal@obry.net>
Cc: Christian Couder <chriscool@tuxfamily.org>,
	git@vger.kernel.org, Theodore Tso <tytso@mit.edu>,
	Jakub Narebski <jnareb@gmail.com>,
	Alex Riesen <raa.lkml@gmail.com>, Andreas Ericsson <ae@op5.se>,
	Matthieu Moy <Matthieu.Moy@imag.fr>,
	Eric Wong <normalperson@yhbt.net>
Subject: Re: [PATCH 1/3] git-help: add -i|--info option to display info page.
Date: Sun, 02 Dec 2007 01:25:42 -0800	[thread overview]
Message-ID: <7vodd9zbvt.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <475272CF.40602@obry.net> (Pascal Obry's message of "Sun, 02 Dec 2007 09:54:39 +0100")

Pascal Obry <pascal@obry.net> writes:

> Christian Couder a écrit :
>> "git help --info XXX" will now call "info git-XXX".
>
> If would be nice if this could be more generic. For example I'd like to
> use Emacs woman mode instead of info. Can't we have something like
>
>    $ git help --ext XXX
>
> "ext" standing for external and calling whatever command recorded into
> .gitconfig for example ?

There is a bit of conflict here.  We could do that and make the
implementation of "ext" command responsible to transform "commit" in

	$ git help --ext commit

to the location of manual page (or formatted HTML page, or node in the
info documentation).  git itself does not need to know much about where
the help material is in such an implementation.

But Christian's series is about making such "ext" thing easier to write.
No matter what kind of web browser is used, it needs to be told where
the preformatted HTML page for git-commit command is (and it does not
care where git-commit.1 manpage is found or what the node is called in
git.info document).  It makes it a bit too limiting by defining -w (web)
and -i (info) upfront without offering -x (ext), but we need to start
somewhere.

Having said that, I think this is a post 1.5.4 material.  Please keep
the discussion going, so that we can have something people can agree on
early after 1.5.4.

  reply	other threads:[~2007-12-02  9:26 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-02  5:07 [PATCH 1/3] git-help: add -i|--info option to display info page Christian Couder
2007-12-02  8:54 ` Pascal Obry
2007-12-02  9:25   ` Junio C Hamano [this message]
2007-12-03  5:53     ` Christian Couder
2007-12-03 18:45       ` Pascal Obry
2007-12-04  5:54         ` Christian Couder

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=7vodd9zbvt.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=Matthieu.Moy@imag.fr \
    --cc=ae@op5.se \
    --cc=chriscool@tuxfamily.org \
    --cc=git@vger.kernel.org \
    --cc=jnareb@gmail.com \
    --cc=normalperson@yhbt.net \
    --cc=pascal@obry.net \
    --cc=raa.lkml@gmail.com \
    --cc=tytso@mit.edu \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.