[GRASS-dev] [GRASS GIS] #3462: Allow setting environment variables in grass startup script

GDAL · January 17, 2018, 8:00am

#3462: Allow setting environment variables in grass startup script
--------------------------+-------------------------
  Reporter: mankoff | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: major | Milestone: 7.2.3
Component: Startup | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: MacOSX
--------------------------+-------------------------

Comment (by martinl):

Done in r72095, please try out fresh trunk.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3462#comment:18>
GRASS GIS <https://grass.osgeo.org>

GDAL · January 17, 2018, 8:01am

#3462: Allow setting environment variables in grass startup script
--------------------------+----------------------
  Reporter: mankoff | Owner: martinl
      Type: defect | Status: assigned
  Priority: major | Milestone: 7.2.3
Component: Startup | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: MacOSX
--------------------------+----------------------
Changes (by martinl):

* status: new => assigned
* cc: grass-dev@… (added)
* owner: grass-dev@… => martinl

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3462#comment:19>
GRASS GIS <https://grass.osgeo.org>

GDAL · February 27, 2018, 9:15pm

#3462: Allow setting environment variables in grass startup script
--------------------------+----------------------
  Reporter: mankoff | Owner: martinl
      Type: defect | Status: assigned
  Priority: major | Milestone: 7.2.3
Component: Startup | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: MacOSX
--------------------------+----------------------

Comment (by martinl):

Does the fix work for you?

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3462#comment:20>
GRASS GIS <https://grass.osgeo.org>

GDAL · February 27, 2018, 9:26pm

#3462: Allow setting environment variables in grass startup script
--------------------------+----------------------
  Reporter: mankoff | Owner: martinl
      Type: defect | Status: assigned
  Priority: major | Milestone: 7.2.3
Component: Startup | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: MacOSX
--------------------------+----------------------

Comment (by mankoff):

Thank you for making the change! I'm sorry I did not thank you and
test/confirm the change previously. However - I cannot test it. I compile
GRASS dev versions on a Linux machine, but because OS X is only partially
supported, I have never gotten a full build/test environment working on OS
X. I need to wait until one of the package managers provides a version
with your updates. I recently upgraded to 7.4.0 but had to downgrade to
7.2.2 because 7.4.0 does not work with QGIS. I *think* you can close this
ticket (again, based on visual inspection of the change).

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3462#comment:21>
GRASS GIS <https://grass.osgeo.org>

GDAL · February 27, 2018, 10:17pm

#3462: Allow setting environment variables in grass startup script
--------------------------+---------------------
  Reporter: mankoff | Owner: martinl
      Type: defect | Status: closed
  Priority: major | Milestone: 7.2.3
Component: Startup | Version: 7.2.2
Resolution: fixed | Keywords:
       CPU: Unspecified | Platform: MacOSX
--------------------------+---------------------
Changes (by neteler):

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

Comment:

Replying to [comment:21 mankoff]:
> Thank you for making the change! I'm sorry I did not thank you and
test/confirm the change previously. However - I cannot test it. I compile
GRASS dev versions on a Linux machine, but because OS X is only partially
supported, I have never gotten a full build/test environment working on OS
X.

Did you try this?
https://grasswiki.osgeo.org/wiki/Compiling_on_MacOSX_using_homebrew

> I need to wait until one of the package managers provides a version with
your updates. I recently upgraded to 7.4.0 but had to downgrade to 7.2.2
because 7.4.0 does not work with QGIS.

It meanwhile does:

https://github.com/qgis/QGIS/pull/6394
* present in QGIS 3.0.0: https://github.com/qgis/QGIS/blob/release-
3_0/cmake/FindGRASS.cmake
* present in QGIS 2.18.17: https://github.com/qgis/QGIS/blob/release-
2_18/cmake/FindGRASS.cmake

> I *think* you can close this ticket (again, based on visual inspection
of the change).

Backport?

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3462#comment:22>
GRASS GIS <https://grass.osgeo.org>

GDAL · February 27, 2018, 10:18pm

#3462: Allow setting environment variables in grass startup script
--------------------------+----------------------
  Reporter: mankoff | Owner: martinl
      Type: defect | Status: reopened
  Priority: major | Milestone: 7.2.3
Component: Startup | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: MacOSX
--------------------------+----------------------
Changes (by neteler):

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

Comment:

Reopen for backport (to 7.4 at least?)

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3462#comment:23>
GRASS GIS <https://grass.osgeo.org>

GDAL · February 28, 2018, 7:31am

#3462: Allow setting environment variables in grass startup script
--------------------------+----------------------
  Reporter: mankoff | Owner: martinl
      Type: defect | Status: reopened
  Priority: major | Milestone: 7.2.3
Component: Startup | Version: 7.2.2
Resolution: | Keywords:
       CPU: Unspecified | Platform: MacOSX
--------------------------+----------------------

Comment (by mankoff):

Did you try this?
https://grasswiki.osgeo.org/wiki/Compiling_on_MacOSX_using_homebrew

I wrote the instructions there for 7.2. At the time the "How to install
GRASS GIS 7.1 SVN Head" section didn't work on 7.2 head. And I'm no longer
using Homebrew but MacPorts - so no I cannot easily test non-packaged
versions on OS X :(.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3462#comment:24>
GRASS GIS <https://grass.osgeo.org>

GDAL · September 3, 2018, 9:23pm

#3462: Allow setting environment variables in grass startup script
-------------------------+-------------------------------------------------
  Reporter: mankoff | Owner: martinl
      Type: defect | Status: closed
  Priority: major | Milestone: 7.6.0
Component: Startup | Version: 7.2.2
Resolution: fixed | Keywords: init grass.py env D_LIBRARY_PATH
       CPU: | bashrc
  Unspecified | Platform: MacOSX
-------------------------+-------------------------------------------------
Changes (by wenzeslaus):

* keywords: => init grass.py env D_LIBRARY_PATH bashrc
* status: reopened => closed
* resolution: => fixed
* milestone: 7.2.4 => 7.6.0

Comment:

This was already fixed and reopened for backport in comment:23. This may
not be a change which is appropriate to backport. At this point, it will
just be in 7.6. Changing milestone. Closing as fixed.

For the record, GRASS GIS itself doesn't use `GRASS_LD_LIBRARY_PATH` at
this point.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3462#comment:27>
GRASS GIS <https://grass.osgeo.org>