[GRASS-dev] select issue with wingrass

[Michael Barton]

Given other discussions about forward and backward slashes, should
env(GISDBASE) be run through file normalize or not? What about env(GISBASE)?

Michael


On 11/18/07 6:49 PM, "Glynn Clements" <glynn at gclements.plus.com> wrote:

> 
> Moritz Lennert wrote:
> 
>>> Ah; the reason it's been working so far is that both lib/gis/gui.tcl
>>> and gui/tcltk/gis.m/gm.tcl set them, e.g.:
>>> 
>>> if {[catch {set env(GISDBASE) [exec g.gisenv get=GISDBASE]} error]} {
>>> 
>>> Any other code which uses select.tcl will need to do likewise.
>>> 
>>> Use of env() is less than ideal (there's already enough confusion
>>> between GRASS variables and environment variables), but using separate
>>> variables would require changing a lot of "global" statements.
>> 
>> Changing above line to
>> 
>> if {[catch {set env(GISDBASE) [exec g.dirseps -g [exec g.gisenv
>> get=GISDBASE]]} error]} {
>> 
>> in lib/gis/gui.tcl seems to work for me. Can I commit ?
>> 
>> Should I also change this in gui/tcltk/gis.m/gm.tcl ?
> 
> Have you tried "file normalize"? If that works, it's preferable to
> spawning g.dirseps.

__________________________________________
Michael Barton, Professor of Anthropology
Director of Graduate Studies
School of Human Evolution & Social Change
Center for Social Dynamics & Complexity
Arizona State University

phone: 480-965-6213
fax: 480-965-7671
www: http://www.public.asu.edu/~cmbarton