Re: [PATCH] git-web--browse: Add support for opera

[Giuseppe Bilotta <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:
> 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