From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 27 Sep 2012 19:11:36 GMT
Subject: Re: [PATCH] git-web--browse: Fix open HTML help pages from iTerm
Message-ID: <7v3923qo0n.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1348597003-28403-1-git-send-email-prohaska@zib.de>

Steffen Prohaska <prohaska@zib.de> writes:

> iTerm is an alternative to the default terminal emulation program on Mac
> OS X.  git-web--browse wasn't aware of iTerm and failed to open HTML
> help pages when used in a shell session running in iTerm, reporting "No
> known browser available."  Now it works as expected.
>
> Signed-off-by: Steffen Prohaska <prohaska@zib.de>
> ---
>  git-web--browse.sh | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/git-web--browse.sh b/git-web--browse.sh
> index 1e82726..95ecf65 100755
> --- a/git-web--browse.sh
> +++ b/git-web--browse.sh
> @@ -120,7 +120,8 @@ if test -z "$browser" ; then
>  	fi
>  	# SECURITYSESSIONID indicates an OS X GUI login session
>  	if test -n "$SECURITYSESSIONID" \
> -		-o "$TERM_PROGRAM" = "Apple_Terminal" ; then
> +		-o "$TERM_PROGRAM" = "Apple_Terminal" \
> +		-o "$TERM_PROGRAM" = "iTerm.app" ; then
>  		browser_candidates="open $browser_candidates"
>  	fi

I do not have anything against iTerm, but could we have a solution
that does not force us to keep adding 47 different terminal program
names to the list over the longer term (no pun intended)?  For
example, "If on OS-X (which by the way does not seem to be checked
with the current logic) and environment TERM_PROGRAM is set to any
value", or something.