[GRASS-dev] Re: [GRASS GIS] #693: wxGUI menus: i.ortho.photo locks up GUI

[GRASS GIS]

#693: wxGUI menus: i.ortho.photo locks up GUI
---------------------+------------------------------------------------------
  Reporter:  hamish  |       Owner:  grass-dev at 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>