[GRASS-dev] [GRASS GIS] #3964: Update browser list to check for chrome

GRASS GRASS developer
1

#3964: Update browser list to check for chrome
-------------------------+-------------------------
Reporter: jidanni | Owner: grass-dev@…
     Type: enhancement | Status: new
Priority: trivial | Milestone:
Component: Startup | Version: unspecified
Keywords: | CPU: Unspecified
Platform: Unspecified |
-------------------------+-------------------------
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.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3964&gt;
GRASS GIS <https://grass.osgeo.org>

2

#3964: Update browser list to check for chrome
--------------------------+-------------------------
  Reporter: jidanni | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: trivial | Milestone:
Component: Startup | Version: unspecified
Resolution: | Keywords:
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------

Comment (by neteler):

Replying to [ticket:3964 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?

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3964#comment:1&gt;
GRASS GIS <https://grass.osgeo.org>

3

#3964: Update browser list to check for chrome
--------------------------+-------------------------
  Reporter: jidanni | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: trivial | Milestone:
Component: Startup | Version: unspecified
Resolution: | Keywords:
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------

Comment (by 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.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3964#comment:2&gt;
GRASS GIS <https://grass.osgeo.org>

4

#3964: Update browser list to check for chrome
--------------------------+-------------------------
  Reporter: jidanni | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: trivial | Milestone:
Component: Startup | Version: unspecified
Resolution: | Keywords:
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------

Comment (by jidanni):

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

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3964#comment:3&gt;
GRASS GIS <https://grass.osgeo.org>

5

#3964: Update browser list to check for chrome
--------------------------+-------------------------
  Reporter: jidanni | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: trivial | Milestone:
Component: Startup | Version: unspecified
Resolution: | Keywords:
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------

Comment (by neteler):

Replying to [comment:1 neteler]:
> https://github.com/OSGeo/grass/pull/201

Merged and backported!

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3964#comment:4&gt;
GRASS GIS <https://grass.osgeo.org>

6

#3964: Update browser list to check for chrome
--------------------------+-------------------------
  Reporter: jidanni | Owner: grass-dev@…
      Type: enhancement | Status: closed
  Priority: trivial | Milestone: 7.8.3
Component: Startup | Version: unspecified
Resolution: fixed | Keywords:
       CPU: Unspecified | Platform: Unspecified
--------------------------+-------------------------
Changes (by wenzeslaus):

* status: new => closed
* resolution: => fixed

Comment:

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

I created a new issue on !GitHub for the `BROWSER` suggestion:
[https://github.com/OSGeo/grass/issues/890 GitHub Issue 890].

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3964#comment:6&gt;
GRASS GIS <https://grass.osgeo.org>