r.li.setup: don't modify the WIND file

[hamish]

Hi,

when you create a sampling window with the mouse in r.li.setup 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.

Hamish

[hamish]

Hi,

Besides adding WIND_OVERRIDE, I have started cleaning up the many unquoted variables in r.li.setup, 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

[neteler]

Replying to hamish:

r.li.setup ... the future of it is for trunk? Was someone working on a python upgrade?

Luca D's prototype is there:

trunk/gui/wxpython/rlisetup

See also: ​http://grass.osgeo.org/grass70/manuals/g.gui.rlisetup.html

[hamish]

Hi, this is now complete in devbr6.

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

One remaining thing to check for G6 r.li.setup 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 r.li.setup tcl code always get written into a temp dir instead of the pwd)

Hamish

[neteler]

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

Closing here, discussion continued in bug #1214.