[GRASS-dev] [GRASS GIS] #1683: error message v.add.col on MacOSX

[GRASS GIS]

#1683: error message v.add.col on MacOSX
---------------------------+------------------------------------------------
 Reporter:  jradinger      |       Owner:  grass-dev@…              
     Type:  defect         |      Status:  new                      
 Priority:  major          |   Milestone:  6.4.3                    
Component:  Shell Scripts  |     Version:  svn-develbranch6         
 Keywords:  v.db.addcol    |    Platform:  All                      
      Cpu:  Unspecified    |  
---------------------------+------------------------------------------------
Changes (by hamish):

  * keywords:  v.add.col => v.db.addcol
  * component:  Database => Shell Scripts


Comment:

 The immediate culprit is `eval g.gisenv`,

 The underlying fix is to not store the addon path as a GRASS variable,
 store it as a shell environment variable -- and only a shell variable --
 instead.

 Remove it from the g.gisenv list with:
 {{{
 g.gisenv set="ADDON_PATH="
 }}}
 then v.db.addcol should be happy.


 Trying to make it both a shell environment variable and a grass variable
 is well intentioned, but blurring definitions so that things can be set
 and read independently in multiple places using the same namespace is
 ultimately going to create more problems than it solves. The main question
 I think is if we need some other place besides .grass.bashrc to store this
 sort of thing, to be sourced by the grass startup script. ? (currently for
 grass6 it needs to be set in ~/.bashrc since .grass.bashrc is sourced
 later in the startup script.

 The GUI can modify its own running environment on the fly with
 os.environ(); command line users who want to modify their environment
 generally can, and indeed want to, take care of themselves.


 cheers,
 Hamish

-- 
Ticket URL: <https://trac.osgeo.org/grass/ticket/1683#comment:5>
GRASS GIS <http://grass.osgeo.org>