git-web--browse: Support for using /bin/start on MinGW

In the future, I think we should also default to xdg-open on Linux instead
of having a KDE-specific hack.

This patch has been sponsored by Novartis.

Signed-off-by: Petr Baudis <pasky@suse.cz>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

diff --git a/Documentation/git-web--browse.txt b/Documentation/git-web--browse.txt
index 7f7a45b2eaa3997cbf5a250fb78387e2a0959a11..278cf735279c13ab129ee28f1ef8e638ccf61dee 100644 (file)
--- a/Documentation/git-web--browse.txt
+++ b/Documentation/git-web--browse.txt
@@ -26,6 +26,7 @@ The following browsers (or commands) are currently supported:
 * lynx
 * dillo
 * open (this is the default under Mac OS X GUI)
 * lynx
 * dillo
 * open (this is the default under Mac OS X GUI)

+* start (this is the default under MinGW)

 
 Custom commands may also be specified.
 
 
 Custom commands may also be specified.
 

diff --git a/git-web--browse.sh b/git-web--browse.sh
index 384148a59fc492d8fb1d6ea4fc4532aa1e5ffc22..78d236b77f6e2b52d89bf7b579762723ede86d15 100755 (executable)
--- a/git-web--browse.sh
+++ b/git-web--browse.sh
@@ -31,7 +31,7 @@ valid_custom_tool()
 
 valid_tool() {
        case "$1" in
 
 valid_tool() {
        case "$1" in

-               firefox | iceweasel | konqueror | w3m | links | lynx | dillo | open)
+               firefox | iceweasel | konqueror | w3m | links | lynx | dillo | open | start)

                        ;; # happy
                *)
                        valid_custom_tool "$1" || return 1
                        ;; # happy
                *)
                        valid_custom_tool "$1" || return 1


@@ -114,6 +114,10 @@ if test -z "$browser" ; then
     if test -n "$SECURITYSESSIONID"; then
        browser_candidates="open $browser_candidates"
     fi
     if test -n "$SECURITYSESSIONID"; then
        browser_candidates="open $browser_candidates"
     fi

+    # /bin/start indicates MinGW
+    if test -n /bin/start; then
+       browser_candidates="start $browser_candidates"
+    fi

 
     for i in $browser_candidates; do
        init_browser_path $i
 
     for i in $browser_candidates; do
        init_browser_path $i


@@ -157,7 +161,7 @@ case "$browser" in
                ;;
        esac
        ;;
                ;;
        esac
        ;;

-    w3m|links|lynx|open)
+    w3m|links|lynx|open|start)

        eval "$browser_path" "$@"
        ;;
     dillo)
        eval "$browser_path" "$@"
        ;;
     dillo)