grass/script/setup.py errors

[artegion]

grass/script/setup.py needs a os.pathsep at this line

    ## os.environ['PATH'] += os.path.join(gisbase, 'lib') ##WRONG##
    os.environ['PATH'] += os.pathsep + os.path.join(gisbase, 'lib')

then $GISBASE/extralib and $GISBASE/msys/bin are missed in PATH (for windows only)

    ## added for winGrass
    if sys.platform.startswith('win'):
         os.environ['PATH'] += os.pathsep + os.path.join(gisbase, 'extralib')
         os.environ['PATH'] += os.pathsep + os.path.join(gisbase, 'msys','bin')

of course we need import sys at the beginning

Another point ---- From outside Grass Gis we must define $GISBASE, $GISRC and add $GISBASE/etc/python to sys.path BEFORE import anything from grass (I did'nt find anything about this in Programmer's Manual)

...
os.environ['GISBASE'] =gisbase                                               
                                                                             
grass_python_dir= os.path.join(gisbase, 'etc','python')                      
if os.path.isdir(os.path.join(grass_python_dir,'grass')):                    
    sys.path.append(grass_python_dir)                                        
else:                                                                        
    raise Exception('etc\python\grass module not found') 
os.environ['GISRC']= 'junk' 
from grass.script import setup                                     
gisrc= setup.init(os.getenv('GISBASE'),gisdb,location,mapset) 

...

os.remove(gisrc)                

[martinl]

Suggested changes applied in r52000 (trunk) and r52001 (devbr6), Backport for relbr64 requires some testing.

Martin

[glynn]

Replying to martinl:

Suggested changes applied in r52000 (trunk)

os.environ['PATH'] += os.pathsep + os.path.join(gisbase, 'msys', 'bin') 

This doesn't belong in GRASS 7.

[martinl]

Replying to glynn:

Replying to martinl:

Suggested changes applied in r52000 (trunk)

> os.environ['PATH'] += os.pathsep + os.path.join(gisbase, 'msys', 'bin') 

This doesn't belong in GRASS 7.

OK, removed in r53313. Closing.

[martinl]

OK, removed in r53313. Closing.

Forgot to mention that the fix has been backported to relbr64 in r53314.