git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
To: git@vger.kernel.org
Cc: Junio C Hamano <gitster@pobox.com>,
	Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Subject: [PATCH 0/6] web--browse cleanups and extensions
Date: Mon, 29 Nov 2010 15:47:49 +0100	[thread overview]
Message-ID: <1291042075-19983-1-git-send-email-giuseppe.bilotta@gmail.com> (raw)

This patches introduces support for Opera, Seamonkey/Iceape and elinks
to git web-browse, after a little reformatting to adhere to the shell
script coding guidelines (guidelines which were not explicit in the
documentation, and which are made so).

The last two patches are somewhat more system-specific (AFAIK, only
Debain & friends have /usr/bin/{x-,}www-browser, and not many other
distributions have chromium under the chromium-browser executable name),
but I believe it still makes sense to include them in the mainstream
version of git.

Giuseppe Bilotta (6):
  CodingGuidelines: mention whitespace preferences for shell scripts
  web--browse: coding style
  web--browse: split valid_tool list
  web--browse: support opera, seamonkey and elinks
  web--browse: use (x-)www-browser if available
  web--browse: special-case chromium path

 Documentation/CodingGuidelines    |    6 +
 Documentation/git-web--browse.txt |   10 ++
 git-web--browse.sh                |  202 +++++++++++++++++++++----------------
 3 files changed, 133 insertions(+), 85 deletions(-)

-- 
1.7.3.2.624.gec6c7.dirty

             reply	other threads:[~2010-11-29 14:48 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-29 14:47 Giuseppe Bilotta [this message]
2010-11-29 14:47 ` [PATCH 1/6] CodingGuidelines: mention whitespace preferences for shell scripts Giuseppe Bilotta
2010-11-30  0:30   ` Junio C Hamano
2010-11-30  8:11     ` Giuseppe Bilotta
2010-11-29 14:47 ` [PATCH 2/6] web--browse: coding style Giuseppe Bilotta
2010-11-29 14:47 ` [PATCH 3/6] web--browse: split valid_tool list Giuseppe Bilotta
2010-11-29 16:44   ` Jonathan Nieder
2010-11-29 19:19     ` Giuseppe Bilotta
2010-11-29 20:24     ` Junio C Hamano
2010-11-29 14:47 ` [PATCH 4/6] web--browse: support opera, seamonkey and elinks Giuseppe Bilotta
2010-11-29 14:47 ` [PATCH 5/6] web--browse: use (x-)www-browser if available Giuseppe Bilotta
2010-11-29 16:18   ` Christian Couder
2010-11-29 16:25     ` Christian Couder
2010-11-29 19:05     ` Giuseppe Bilotta
2010-11-30  4:02       ` Christian Couder
2010-11-30  8:22         ` Giuseppe Bilotta
2010-12-01 10:59           ` Christian Couder
2010-12-01 11:46             ` Giuseppe Bilotta
2010-12-01 15:49               ` Jonathan Nieder
2010-11-29 16:48   ` Jonathan Nieder
2010-11-29 19:16     ` Giuseppe Bilotta
2010-11-29 14:47 ` [PATCH 6/6] web--browse: special-case chromium path Giuseppe Bilotta

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=1291042075-19983-1-git-send-email-giuseppe.bilotta@gmail.com \
    --to=giuseppe.bilotta@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    /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).