Re: [PATCH] git-web--browse: Add support for opera
- From
Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
- Date
- Nov 27, 2010, 15:19 UTC
- Message-ID
- <AANLkTik-PnsZ7tBEgLixoKOYYrytdOGfDocuMECGn-6O@mail.gmail.com>
- In-Reply-To
- <1290856558-6122-1-git-send-email-giuseppe.bilotta@gmail.com>
Actually, ignore this patch for the time being, I'm preparing a longer set that includes it but might need some discussion. Sorry for the noise.
On Sat, Nov 27, 2010 at 12:15 PM, Giuseppe Bilotta <giuseppe.bilotta@gmail.com> wrote:
Show 44 quoted lines
> It's probably not the most widespread browser, it's free as in gratis
> but not as in libre, but it's worth supporting regardless, especially
> since it's a trivial addition.
>
> Signed-off-by: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
> ---
> git-web--browse.sh | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/git-web--browse.sh b/git-web--browse.sh
> index 3fc4166..eb70375 100755
> --- a/git-web--browse.sh
> +++ b/git-web--browse.sh
> @@ -31,7 +31,7 @@ valid_custom_tool()
>
> valid_tool() {
> case "$1" in
> - firefox | iceweasel | chrome | google-chrome | chromium | konqueror | w3m | links | lynx | dillo | open | start)
> + firefox | iceweasel | chrome | google-chrome | chromium | konqueror | opera | w3m | links | lynx | dillo | open | start)
> ;; # happy
> *)
> valid_custom_tool "$1" || return 1
> @@ -103,7 +103,7 @@ fi
>
> if test -z "$browser" ; then
> if test -n "$DISPLAY"; then
> - browser_candidates="firefox iceweasel google-chrome chrome chromium konqueror w3m links lynx dillo"
> + browser_candidates="firefox iceweasel google-chrome chrome chromium konqueror opera w3m links lynx dillo"
> if test "$KDE_FULL_SESSION" = "true"; then
> browser_candidates="konqueror $browser_candidates"
> fi
> @@ -173,7 +173,7 @@ case "$browser" in
> start)
> exec "$browser_path" '"web-browse"' "$@"
> ;;
> - dillo)
> + opera|dillo)
> "$browser_path" "$@" &
> ;;
> *)
> --
> 1.7.3.2.184.gecaee
>
>-- Giuseppe "Oblomov" Bilotta