git/git-web--browse.sh, branch jch

git-web--browse.sh: use the $( ... ) construct for command substitution

2014-04-23T22:17:02Z

The Git CodingGuidelines prefer the $(...) construct for command substitution instead of using the backquotes `...`. The backquoted form is the traditional method for command substitution, and is supported by POSIX. However, all but the simplest uses become complicated quickly. In particular, embedded command substitutions and/or the use of double quotes require careful escaping with the backslash character. The patch was generated by: for _f in $(find . -name "*.sh") do sed -i 's@`$.*$`@$(\1)@g' ${_f} done and then carefully proof-read. Signed-off-by: Elia Pinto Reviewed-by: Matthieu Moy Signed-off-by: Junio C Hamano

web--browse: Add support for xdg-open

2013-10-30T20:54:15Z

xdg-open is a tool similar to git-web--browse. It opens a file or URL in the user's preferred application. It could probably be made default at least on Linux with a graphical environment. Signed-off-by: Rüdiger Sonderfeld Signed-off-by: Junio C Hamano

web--browse: support /usr/bin/cygstart on Cygwin

2013-06-21T16:05:15Z

While both GUI and console Cygwin browsers do exist, anecdotal evidence suggests most users rely on their native Windows browser. cygstart, which is a long-standing part of the base Cygwin installation, will cause the page to be opened in the default Windows browser (the one registered to open .html files). Signed-off-by: Yaakov Selkowitz Signed-off-by: Junio C Hamano

git-web--browse: recognize any TERM_PROGRAM as a GUI terminal on OS X

2013-03-29T22:24:31Z

It turns out that the presence of SECURITYSESSIONID is not sufficient for detecting the presence of a GUI under Mac OS X. SECURITYSESSIONID appears to only be set when the user has Screen Sharing enabled. Disabling Screen Sharing and relaunching the shell showed that the variable was missing, at least under Mac OS X 10.6.8. On the other hand, TERM_PROGRAM seems to be set for any terminals on OS X, so just check it is set to something, instead of hardcoding "Apple_Terminal" and missing other terminals such as iTerm.app. Signed-off-by: John Szakmeister Signed-off-by: Junio C Hamano

git-web--browse: avoid the use of eval

2011-10-03T17:47:07Z

Using eval causes problems when the URL contains an appropriately escaped ampersand (\&). Dropping eval from the built-in browser invocation avoids the problem. Helped-by: Jeff King (test case) Signed-off-by: Chris Packham Signed-off-by: Junio C Hamano

web--browse: better support for chromium

2010-12-03T22:05:32Z

On Debian-based distributions, Chromium the browser is available under the name chromium-browser rather than chromium, to prevent conflicts with the Chromium B.S.U. game. Look for chromium-browser first when setting the path for chromium, and also add chromium-browser as a supported browser name. Document the dual-name support, and mention the dual-name support for (google-)chrome too. Signed-off-by: Giuseppe Bilotta Signed-off-by: Junio C Hamano

web--browse: support opera, seamonkey and elinks

2010-12-03T21:49:14Z

The list of supported browsers is also updated in the documentation. Signed-off-by: Giuseppe Bilotta Signed-off-by: Junio C Hamano

web--browse: split valid_tool list

2010-12-03T21:46:24Z

It was getting too long, and we want to add some more. Signed-off-by: Giuseppe Bilotta Signed-off-by: Junio C Hamano

web--browse: coding style

2010-12-03T21:44:07Z

Retab and deindent choices in case statements. Signed-off-by: Giuseppe Bilotta Signed-off-by: Junio C Hamano

Add a google-chrome option for web--browse

2010-07-26T06:14:18Z

On some systems, the chrome browser is named google-chrome. We add support for this case. Signed-off-by: Nathan W. Panike Signed-off-by: Junio C Hamano