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@xxxxxxxxx> 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@xxxxxxxxx> > --- > 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html