{"thread":{"id":"24470","subject":"[PATCH] Add a google-chrome option for web--browse","startedAt":"2010-07-22T16:58:20Z","lastAt":"2010-07-22T16:58:20Z","messageCount":1,"participants":["Nathan W. Panike"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"146001","messageId":"4c487a2a.4814e70a.2020.332e@mx.google.com","threadId":"24470","inReplyTo":null,"subject":"[PATCH] Add a google-chrome option for web--browse","fromName":"Nathan W. Panike","fromEmail":"nathan.panike@gmail.com","sentAt":"2010-07-22T16:58:20Z","receivedAt":"2010-07-22T16:58:20Z","isPatch":true,"sender":{"key":"nathan.panike@gmail.com","avatar":"https://avatars.githubusercontent.com/u/389447?v=4"},"body":"On some systems, the chrome browser is named google-chrome.  We add\nsupport for this case.\n\nSigned-off-by: Nathan W. Panike <nathan.panike@gmail.com>\n---\nI have been downloading the google-chrome-beta .deb package from Google, and\nthe binary is named google-chrome from that package.\n\n git-web--browse.sh |    6 +++---\n 1 files changed, 3 insertions(+), 3 deletions(-)\n\ndiff --git a/git-web--browse.sh b/git-web--browse.sh\nindex dbded76..3fc4166 100755\n--- a/git-web--browse.sh\n+++ b/git-web--browse.sh\n@@ -31,7 +31,7 @@ valid_custom_tool()\n \n valid_tool() {\n \tcase \"$1\" in\n-\t\tfirefox | iceweasel | chrome | chromium | konqueror | w3m | links | lynx | dillo | open | start)\n+\t\tfirefox | iceweasel | chrome | google-chrome | chromium | konqueror | w3m | links | lynx | dillo | open | start)\n \t\t\t;; # happy\n \t\t*)\n \t\t\tvalid_custom_tool \"$1\" || return 1\n@@ -103,7 +103,7 @@ fi\n \n if test -z \"$browser\" ; then\n     if test -n \"$DISPLAY\"; then\n-\tbrowser_candidates=\"firefox iceweasel chrome chromium konqueror w3m links lynx dillo\"\n+\tbrowser_candidates=\"firefox iceweasel google-chrome chrome chromium konqueror w3m links lynx dillo\"\n \tif test \"$KDE_FULL_SESSION\" = \"true\"; then\n \t    browser_candidates=\"konqueror $browser_candidates\"\n \tfi\n@@ -146,7 +146,7 @@ case \"$browser\" in\n \ttest \"$vers\" -lt 2 && NEWTAB=''\n \t\"$browser_path\" $NEWTAB \"$@\" &\n \t;;\n-    chrome|chromium)\n+    google-chrome|chrome|chromium)\n \t# Actual command for chromium is chromium-browser.\n \t# No need to specify newTab. It's default in chromium\n \teval \"$browser_path\" \"$@\" &\n-- \n1.7.2\n"}]}