From: Matthieu Moy <Matthieu.Moy@imag.fr>
To: Andreas Ericsson <ae@op5.se>
Cc: Alex Riesen <raa.lkml@gmail.com>,
Christian Couder <chriscool@tuxfamily.org>,
Junio Hamano <junkio@cox.net>,
git@vger.kernel.org, Theodore Tso <tytso@mit.edu>
Subject: Re: [RFC/PATCH] git-help: add new options -w (for web) and -i (for info)
Date: Sat, 24 Nov 2007 19:23:10 +0100 [thread overview]
Message-ID: <vpqlk8nv6y9.fsf@bauges.imag.fr> (raw)
In-Reply-To: <47482116.70600@op5.se> (Andreas Ericsson's message of "Sat\, 24 Nov 2007 14\:03\:18 +0100")
Andreas Ericsson <ae@op5.se> writes:
> Alex Riesen wrote:
>> Christian Couder, Sat, Nov 24, 2007 05:16:50 +0100:
>>> +valid_tool() {
>>> + case "$1" in
>>> + firefox | iceweasel | konqueror | w3m)
>>> + ;; # happy
>>
>> lynx, links and dillo?
>>
>
> Why bother validating it at all? I might be using a script to
> add some formatting and then pass it through the browser of my
> choice. A friend of mine who's visually impaired does just that
> to strip some CSS tags which can make docs quite unreadable for
> him.
In any case, make sure it's consistant with git instaweb, which also
runs a web browser (firefox by default, overridable with --browser or
instaweb.browser).
--
Matthieu
prev parent reply other threads:[~2007-11-24 18:23 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-24 4:16 [RFC/PATCH] git-help: add new options -w (for web) and -i (for info) Christian Couder
2007-11-24 8:52 ` Jakub Narebski
2007-11-24 11:34 ` Alex Riesen
2007-11-24 13:03 ` Andreas Ericsson
2007-11-24 17:33 ` Alex Riesen
2007-11-24 18:23 ` Matthieu Moy [this message]
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=vpqlk8nv6y9.fsf@bauges.imag.fr \
--to=matthieu.moy@imag.fr \
--cc=ae@op5.se \
--cc=chriscool@tuxfamily.org \
--cc=git@vger.kernel.org \
--cc=junkio@cox.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox