diff options
| author | Jesse Stuart | 2015-11-16 12:29:17 -0600 |
|---|---|---|
| committer | Jesse Stuart | 2015-11-16 12:29:17 -0600 |
| commit | 1111d28c951aa8f5816b080a48658dde7ab02bb8 (patch) | |
| tree | 2992d99464370f069fbee8b56d0dca6761f95319 | |
| parent | 3bc3fa0b3a3b0f1b6273531d4026fee46443eb67 (diff) | |
| download | browser-1111d28c951aa8f5816b080a48658dde7ab02bb8.tar.bz2 | |
Don't use `sensible-browser` directly (linux)
`sensible-browser` blocks the current process and doesn't open the
expected browser (using system vs user default).
`xdg-open` is non-blocking, uses the user default browser, and falls
back to using `sensible-browser`.
See https://github.com/flynn/flynn/issues/2125#issuecomment-156953583
Signed-off-by: Jesse Stuart <jesse@jessestuart.ca>
| -rw-r--r-- | browser_linux.go | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/browser_linux.go b/browser_linux.go index b1b40e8..bed47dd 100644 --- a/browser_linux.go +++ b/browser_linux.go @@ -1,10 +1,5 @@ package browser func openBrowser(url string) error { - // try sensible-browser first - if err := runCmd("sensible-browser", url); err == nil { - return nil - } - // sensible-browser not availble, try xdg-open return runCmd("xdg-open", url) } |