[GRASS-dev] [GRASS GIS] #1181: r.le.setup fails on WinGRASS due to missing xterm

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------+-----------------------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.1
Component: wxGUI | Version: svn-releasebranch64
Keywords: | Platform: MSWindows XP
      Cpu: All |
----------------------+-----------------------------------------------------
In the most recent 6.4 nightly build
http://josef.fsv.cvut.cz/wingrass/grass64/WinGRASS-6.4.SVN-r43715-1-Setup.exe
r.le.setup called from wxGUI fails in following way:

{{{
Traceback (most recent call last):
   File "c:/Program
Files/GRASS-64-SVN/etc/wxpython/wxgui.py", line 1011, in
OnXTerm

del os.environ['GRASS_RENDER_IMMEDIATE']
   File "C:\Program Files\GRASS-64-SVN\Python25\lib\os.py",
line 441, in __delitem__

del self.data[key.upper()]
KeyError
:
'GRASS_RENDER_IMMEDIATE'
}}}

On Debian Squeeze, built from r43697 of releasebranch_6_4, it starts, but
some error crops out too:

{{{
Traceback (most recent call last):
   File "/usr/lib64/python2.6/dist-
packages/wx-2.8-gtk2-unicode/wx/lib/agw/flatnotebook.py",
line 3989, in OnLeftDown

if self._pagesInfoVec[tabIdx].GetEnabled():
IndexError
:
list index out of range
}}}

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1181&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------------------+-----------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.1
Component: wxGUI | Version: svn-releasebranch64
Keywords: wingrass, r.le.setup | Platform: MSWindows XP
      Cpu: All |
----------------------------------+-----------------------------------------
Changes (by hamish):

  * keywords: => wingrass, r.le.setup

Comment:

afaik r.le.* on wingrass should have been greyed out and unavailable.

?

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1181#comment:1&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------------------+-----------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.1
Component: wxGUI | Version: svn-releasebranch64
Keywords: wingrass, r.le.setup | Platform: MSWindows XP
      Cpu: All |
----------------------------------+-----------------------------------------

Comment(by hellik):

Replying to [comment:1 hamish]:
> afaik r.le.* on wingrass should have been greyed out and unavailable.
>
> ?

WinGrass6.4.svn: all r.li-stuff is greyed out, but not the r.le-stuff.

Helmut

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1181#comment:2&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------------------+-----------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.1
Component: wxGUI | Version: svn-releasebranch64
Keywords: wingrass, r.le.setup | Platform: MSWindows XP
      Cpu: All |
----------------------------------+-----------------------------------------

Comment(by hamish):

ah, right. crossed wires and/or low blood sugar at this end..

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1181#comment:3&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------------------+-----------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.1
Component: wxGUI | Version: svn-releasebranch64
Keywords: wingrass, r.le.setup | Platform: MSWindows XP
      Cpu: All |
----------------------------------+-----------------------------------------

Comment(by hellik):

Replying to [comment:3 hamish]:
> ah, right. crossed wires and/or low blood sugar at this end..

r.le.setup requires xterm (indicated in the menue). is this available on
windows?

but very interesting, if you type r.le.setup in the wx-gui-command-line or
in the msys-command-line, the r.le.setup-wx-gui pops up.

Helmut

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1181#comment:4&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------------------+-----------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.1
Component: wxGUI | Version: svn-releasebranch64
Keywords: wingrass, r.le.setup | Platform: MSWindows XP
      Cpu: All |
----------------------------------+-----------------------------------------

Comment(by hamish):

r.le.setup and r.le.trace require an Xmon to function, and will not work
in native WinGrass. Suggest that it be grey'd out.

AFAIK both r.le.* (requires xmons) and r.li.* (requires unix sockets) will
work in the cygwin build.

Hamish

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1181#comment:5&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------------------+-----------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: svn-releasebranch64
Keywords: wingrass, r.le.setup | Platform: MSWindows XP
      Cpu: All |
----------------------------------+-----------------------------------------
Changes (by hamish):

  * milestone: 6.4.1 => 6.4.3

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1181#comment:6&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------------------+-----------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: svn-releasebranch64
Keywords: wingrass, r.le.setup | Platform: MSWindows XP
      Cpu: All |
----------------------------------+-----------------------------------------

Comment(by hamish):

ghelp.py pops up a message explaining that you have to run the command
from the terminal for OnXTerm.

how to detect if we're on win32 but running cygwin?

currently the native wingrass error for r.le.setup is:
{{{
Traceback...
    File "c:\program files\grass ...\wxpython\lmgr\frame.py" line 1148, in
OnXTerm

xmon = xmonlist[0]
IndexError
:
list index out of range
}}}

seems like that would be reasonably easy to catch, and it should do, but
it's pretty easy to just not build the two problematic programs if X11 is
not present. (done in devbr6 in r50699, please test)

technically r.le.setup is interactive on the Xmon and interactive in the
terminal; and r.le.trace draws to the graphics driver but is not
interactive with it, but does use G_yes() so needs to run from an
interactive terminal.

the r.li ones are all successfully greyed out as they were never built.

r.digit and i.class are not built, so greyed out successfully.

g.setproj on native WinGrass fails with a Traceback error (KerError in
OnXTerm: `del self.data[key.upper()]` Global name 'Command' is not
defined); but should be present and should work from the command prompt.

Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1181#comment:7&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
----------------------------------+-----------------------------------------
Reporter: msieczka | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI | Version: svn-releasebranch64
Keywords: wingrass, r.le.setup | Platform: MSWindows XP
      Cpu: All |
----------------------------------+-----------------------------------------

Comment(by hamish):

r.le vs. wingrass patches backported to relbr64 in r50703.

all OnXTerm menu items seem broken currently in 6.5svn wxgui (linux too)
but seem to work in 6.4svn.

Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1181#comment:8&gt;
GRASS GIS <http://grass.osgeo.org>

#1181: r.le.setup fails on WinGRASS due to missing xterm
---------------------------+------------------------------------------------
  Reporter: msieczka | Owner: grass-dev@…
      Type: defect | Status: closed
  Priority: normal | Milestone: 6.4.2
Component: wxGUI | Version: svn-releasebranch64
Resolution: fixed | Keywords: wingrass, r.le.setup
  Platform: MSWindows XP | Cpu: All
---------------------------+------------------------------------------------
Changes (by hamish):

  * status: new => closed
  * resolution: => fixed
  * milestone: 6.4.3 => 6.4.2

Comment:

closing bug as fixed. OnXTerm breakage in 6.5svn continued in #945.

Hamish

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/1181#comment:9&gt;
GRASS GIS <http://grass.osgeo.org>