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

Giuseppe Bilotta· Nov 27, 2010, 11:15 UTC · lore

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(-)

Show changes to git-web--browse.sh +3 −3

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 Bilotta· Nov 27, 2010, 15:19 UTC · re: Giuseppe Bilotta · lore

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

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

patchgit-web--browse: Add support for opera