Hamish wrote:
> looking into why it isn't like {|^GISDBASE: \(.*\)$} already with
> 'svn blame', looks like r45817 introduced it. Maris, do you remember
> what the problem is that was fixing, so we can avoid reintroducing it?
?
re. patch for scripts within a script not working:
> any comments before committing it?
Replying to [comment:17 hamish]:
> it's the `GRASS GIS 6.x Old TclTk GUI` startup menu item which is eating
the C: part of GISDBASE.
>
> specifically this line in lib/init/gis_set.tcl:
{{{
if { [regexp -- {^GISDBASE.*:(.*)$} $thisline dummy env_database] } {
set database [string trim $env_database]
}
}}}
>
> looking into why it isn't like `{|^GISDBASE: \(.*\)$}` already with
> 'svn blame', looks like r45817 introduced it. Maris, do you remember
> what the problem is that was fixing, so we can avoid reintroducing it?
it was a work-around for #1340, which was fixed a long time ago.
partial revert of r45817 done in devbr6 with r56071.
* script-called-from-script (adding \scripts to %PATH%) done & tested in
devbr6 r56092 (cmd.exe and tcl/tk gui startups) and backported to relbr64
in r56104 (still to be tested there).
* tcl/tk gui eating C:\ from GISDBASE done in grassrc6 in r56071 & tested
for devbr6, backported to relbr64 in r56105 (still to be tested there).
* script-called-from-script in cmd.exe and tcl/tk wingrass startups
* .bat wrappers for shell scripts now have DOS newlines
* tcl/tk eating C:\ from GISDBASE in grassrc6