Skip to content

Commit

Permalink
Add sensible-browser to the browser list
Browse files Browse the repository at this point in the history
Users on Debian-based systems who are running without a GUI might still have a text based web browser such as w3m or lynx. Debian gives us the sensible-browser command to launch one of these (either by environment variable or with the alternatives system). xdg-open still comes first, since GUI users will be less inclined to configure the sensible utils.
  • Loading branch information
jleedev authored Apr 28, 2017
1 parent 7b3e8fb commit 70b01bc
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/rustup-utils/src/raw.rs
Original file line number Diff line number Diff line change
Expand Up @@ -367,7 +367,7 @@ pub fn find_cmd<'a>(cmds: &[&'a str]) -> Option<&'a str> {
pub fn open_browser(path: &Path) -> io::Result<bool> {
#[cfg(not(windows))]
fn inner(path: &Path) -> io::Result<bool> {
let commands = ["xdg-open", "open", "firefox", "chromium"];
let commands = ["xdg-open", "open", "firefox", "chromium", "sensible-browser"];
if let Some(cmd) = find_cmd(&commands) {
Command::new(cmd)
.arg(path)
Expand Down

0 comments on commit 70b01bc

Please sign in to comment.