Opened 13 years ago

Closed 9 years ago

#721 closed defect (fixed) don't modify the WIND file

Reported by: hamish Owned by: grass-dev@…
Priority: normal Milestone: 6.4.4
Component: Raster Version: 6.4.0 RCs
Keywords:, g.region, Cc:
CPU: All Platform: All



when you create a sampling window with the mouse in it changes the current region. It should act like v.digit (tcl version) and only do that internally/in memory or by using GRASS_REGION or WIND_OVERRIDE.


Change History (5)

comment:1 by hamish, 11 years ago

Keywords: g.region added

comment:2 by hamish, 10 years ago

Keywords: added


Besides adding WIND_OVERRIDE, I have started cleaning up the many unquoted variables in, and replacing tiny tmp files with shell script backticks, but before going too far wonder what the future of it is for trunk? Was someone working on a python upgrade?

I'd note it uses Xmons, shell scripts, tcltk, and IIRC unix sockets(?), so not especially tuned for grass7.

thanks, Hamish

in reply to:  2 comment:3 by neteler, 10 years ago

Replying to hamish: ... the future of it is for trunk? Was someone working on a python upgrade?

Luca D's prototype is there:


See also:

comment:4 by hamish, 9 years ago


Hi, this is now complete in devbr6.

trunk is continued in #421. backport to relbr64 is todo.

One remaining thing to check for G6 is if there are places in the tcl code which need "quoting" around filename variables which may contain spaces. (e.g. path to temp files) If regular tcl variables are ok(?) there are still some exec statements which could need checking.

(Another thing to check is that temp files in the tcl code always get written into a temp dir instead of the pwd)


comment:5 by neteler, 9 years ago

Resolution: fixed
Status: newclosed

AFAIK fixed in the rewrite in trunk (backported to G6 in r59304 and r59305).

Closing here, discussion continued in bug #1214.

Note: See TracTickets for help on using tickets.