git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] git-web--browse: Fix open HTML help pages from iTerm

From
Steffen Prohaska <prohaska@zib.de>
Date
Sep 27, 2012, 19:31 UTC
Message-ID
<0D4FA057-D2C1-4235-961F-166FD9DCA4F4@zib.de>
In-Reply-To
<7v3923qo0n.fsf@alter.siamese.dyndns.org>
On Sep 27, 2012, at 9:11 PM, Junio C Hamano wrote:
Show 32 quoted lines
> 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.

I googled a bit and it seems that TERM_PROGRAM is specific to OS X. So simply testing whether TERM_PROGRAM is set to any value (without additional check for OS X) might be good enough.

I am wondering whether anyone knows if TERM_PROGRAM is used on other operating systems besides OS X.

	Steffen
Previous: Junio C Hamano
Message 3 of 3 in “git-web--browse: Fix open HTML help pages from iTerm”
  1. git-web--browse: Fix open HTML help pages from iTermSteffen Prohaska, Sep 25, 2012
  2. Junio C HamanoSep 27, 2012
  3. Steffen ProhaskaSep 27, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.