Opened 14 years ago

Closed 13 years ago

#1181 closed defect (fixed)

r.le.setup fails on WinGRASS due to missing xterm

Reported by: msieczka Owned by: grass-dev@…
Priority: normal Milestone: 6.4.2
Component: wxGUI Version: svn-releasebranch64
Keywords: wingrass, r.le.setup Cc:
CPU: All Platform: MSWindows XP

Description

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

Change History (9)

comment:1 by hamish, 14 years ago

Keywords: wingrass r.le.setup added

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

?

in reply to:  1 comment:2 by hellik, 14 years ago

Replying to 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

comment:3 by hamish, 14 years ago

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

in reply to:  3 comment:4 by hellik, 14 years ago

Replying to 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

comment:5 by hamish, 13 years ago

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

comment:6 by hamish, 13 years ago

Milestone: 6.4.16.4.3

comment:7 by hamish, 13 years ago

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

comment:8 by hamish, 13 years ago

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

comment:9 by hamish, 13 years ago

Milestone: 6.4.36.4.2
Resolution: fixed
Status: newclosed

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

Hamish

Note: See TracTickets for help on using tickets.