[WSL] Open in browser does not work

[ishaan-kapoor]

I am using bash with vi mode, in wsl and $BROWSER env variable is set.
When I press Ctrl+O, nothing happens, I also treid changing BROWSER to vim and still nothing changed.

[junegunn]

We use open on macOS and xdg-open in other systems to open URLs.

fzf-git.sh/fzf-git.sh

Lines 107 to 110 in 23a74ef

	case "$(uname -s)" in
	Darwin) open "$url$path" ;;
	*) xdg-open "$url$path" ;;
	esac

Maybe xdg-open is not available on your system? Please open a pull request if a different command is required.

[ishaan-kapoor]

I use WSL and I have replaced the switch-case statement with the following line in my local copy

$BROWSER"$url$path"

And the BROWSER environment variable is set to 'explorer.exe microsoft-edge:'.

Note: There is no space between $BROWSER and "$url$path"

Should I identify WSL by uname -r and add a PR?

[junegunn]

A pull request is welcome.

By the way, I'm not sure why you're mentioning $BROWSER. This project doesn't use the variable.

[ishaan-kapoor]

I am using the shell variable $BROWSER (set in my .bashrc) and as per this thread on SO, it is common practice.

[junegunn]

it is common practice

Maybe, but not on this project.

Note: There is no space between $BROWSER and "$url$path"

The examples shown in the linked SO thread all seem to require a space after it to work with arguments. e.g. export BROWSER='/home/username/lynx.sh'. So even if a program implements $BROWSER scheme, it will unlikely work for you as it will run something like $BROWSER "$url".

[ishaan-kapoor]

It's a WSL issue, more specifically, I am using the browser form my native windows, and accessing it through WSL.
It seems like this is not a common practice as a minority of users have this configuration and this might create bugs for other WSL users who use browsers installed in WSL itself.