Skip to content

Commit 48e50e3

Browse files
committed
Auto merge of #1087 - jleedev:patch-1, r=brson
Add sensible-browser to the browser list 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. This will also allow running the GUI web browser in a non-xdg system that doesn't specifically have firefox or chromium.
2 parents d330705 + 70b01bc commit 48e50e3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/rustup-utils/src/raw.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -367,7 +367,7 @@ pub fn find_cmd<'a>(cmds: &[&'a str]) -> Option<&'a str> {
367367
pub fn open_browser(path: &Path) -> io::Result<bool> {
368368
#[cfg(not(windows))]
369369
fn inner(path: &Path) -> io::Result<bool> {
370-
let commands = ["xdg-open", "open", "firefox", "chromium"];
370+
let commands = ["xdg-open", "open", "firefox", "chromium", "sensible-browser"];
371371
if let Some(cmd) = find_cmd(&commands) {
372372
Command::new(cmd)
373373
.arg(path)

0 commit comments

Comments
 (0)