34 lines
1.1 KiB
Diff
34 lines
1.1 KiB
Diff
From e44161a223e7ec2cfb1f1b71399b995626c2d208 Mon Sep 17 00:00:00 2001
|
|
Message-Id: <e44161a223e7ec2cfb1f1b71399b995626c2d208.1234127422.git.tmz@pobox.com>
|
|
From: Todd Zullinger <tmz@pobox.com>
|
|
Date: Sun, 8 Feb 2009 14:04:20 -0500
|
|
Subject: [PATCH 1/2] git-web--browse: Fix check for /bin/start
|
|
|
|
The previous check in git-web--browse for /bin/start used test -n
|
|
/bin/start, which was always true. This lead to "start" being tried
|
|
first in the browser list. On systems with upstart installed, "start"
|
|
exists and might be in the PATH, but it makes a poor choice for a web
|
|
browser. Instead, test that /bin/start exists and is executable.
|
|
|
|
Signed-off-by: Todd Zullinger <tmz@pobox.com>
|
|
---
|
|
git-web--browse.sh | 2 +-
|
|
1 files changed, 1 insertions(+), 1 deletions(-)
|
|
|
|
diff --git a/git-web--browse.sh b/git-web--browse.sh
|
|
index 78d236b..7ed0fad 100755
|
|
--- a/git-web--browse.sh
|
|
+++ b/git-web--browse.sh
|
|
@@ -115,7 +115,7 @@ if test -z "$browser" ; then
|
|
browser_candidates="open $browser_candidates"
|
|
fi
|
|
# /bin/start indicates MinGW
|
|
- if test -n /bin/start; then
|
|
+ if test -x /bin/start; then
|
|
browser_candidates="start $browser_candidates"
|
|
fi
|
|
|
|
--
|
|
1.6.1.3
|
|
|