rpms/git/devel 0001-git-web-browse-Fix-check-for-bin-start.patch, NONE, 1.1

Todd M. Zullinger tmz at fedoraproject.org
Mon Feb 9 06:40:44 UTC 2009


Author: tmz

Update of /cvs/extras/rpms/git/devel
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv17021

Added Files:
	0001-git-web-browse-Fix-check-for-bin-start.patch 
Log Message:
Add git-web--browse patch

0001-git-web-browse-Fix-check-for-bin-start.patch:

--- NEW FILE 0001-git-web-browse-Fix-check-for-bin-start.patch ---
>From e44161a223e7ec2cfb1f1b71399b995626c2d208 Mon Sep 17 00:00:00 2001
Message-Id: <e44161a223e7ec2cfb1f1b71399b995626c2d208.1234127422.git.tmz at pobox.com>
From: Todd Zullinger <tmz at 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 at 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





More information about the fedora-extras-commits mailing list