#693: wxGUI menus: i.ortho.photo locks up GUI
---------------------+------------------------------------------------------
Reporter: hamish | Owner: grass-dev@lists.osgeo.org
Type: defect | Status: new
Priority: major | Milestone: 6.4.0
Component: wxGUI | Version: svn-develbranch6
Resolution: | Keywords: i.ortho.photo
Platform: All | Cpu: x86-64
---------------------+------------------------------------------------------
Comment (by glynn):
Replying to [comment:14 hamish]:
> I'm not sure of the RENDER_IMMEDIATE directly after. I assume it just
has to be set for the launch of the command, and as soon as it is forked
it can be set back.
Yes. Although modifying os.environ is really the wrong approach. It's
preferable to pass a modified version via env=, e.g.:
{{{
tmpenv = os.environ.copy()
tmpenv['GRASS_RENDER_IMMEDIATE'] = 'FALSE'
p = Popen(..., env = tmpenv)
}}}
Then you don't have to worry about os.environ not getting restored if an
exception is raised.
> no idea if the WinNT version works or is even meaningful, as d.mon will
have failed there ? Or does Cygwin will call its os.environ as Windows_NT
so for that it is needed? I guess we could catch the d.mon call but really
r.digit, i.ortho.photo, etc should be greyed out on native WinGrass before
then.
Cygwin will leave OS=Windows_NT alone. But Cygwin should be using the
xterm version not the cmd.exe version.
But I think that this is an issue of not distinguishing between xterm+xmon
and just xterm.
--
Ticket URL: <https://trac.osgeo.org/grass/ticket/693#comment:19>
GRASS GIS <http://grass.osgeo.org>
_______________________________________________
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev
