Update browser list to check for chrome

[jidanni]

​https://github.com/OSGeo/grass/blob/master/lib/init/grass.py#L678 mentions firefox, but not chrome or chromium.

And maybe it needs to do os.getenv('BROWSER') also.

[neteler]

Replying to jidanni:

​https://github.com/OSGeo/grass/blob/master/lib/init/grass.py#L678 mentions firefox, but not chrome or chromium.

Ok, please try

​https://github.com/OSGeo/grass/pull/201

And maybe it needs to do os.getenv('BROWSER') also.

No idea, do other devs have an opinion?

[jidanni]

Everybody knows about $BROWSER.

$ w3m -dump https://wiki.archlinux.org/index.php/Environment_variables | grep -C 2 BROWSER

  • BROWSER contains the path to the web browser. Helpful to set in an
    interactive shell configuration file so that it may be dynamically altered
    depending on the availability of a graphic environment, such as X:

if [ -n "$DISPLAY" ]; then
    export BROWSER=firefox
else
    export BROWSER=links
fi

​https://unix.stackexchange.com/a/213369/179494 etc.

[jidanni]

​https://askubuntu.com/questions/27447/google-earth-and-browser-environment-variable

[neteler]

Replying to neteler:

​https://github.com/OSGeo/grass/pull/201

Merged and backported!

[wenzeslaus]

The Chrome/Chromium part was resolved in ​PR:201, so closing as fixed.

I created a new issue on GitHub for the BROWSER suggestion: ​GitHub Issue 890.