Re: [GRASS-dev] [GRASS GIS] #1950: wingrass: replace rxvt with conhost.exe
#1950: wingrass: replace rxvt with conhost.exe ---+ Reporter: hamish | Owner: grass-dev@… Type: task | Status: new Priority: major | Milestone: 6.4.4 Component: Packaging | Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows 7 Cpu: All| ---+ Comment(by hamish): Replying to [ticket:1950 hamish]: IIUC, conhost.exe is new for Windows7, Windows XP doesn't have it. At least mine doesn't in C:\windows\system32. maybe it is csrss.exe there? -- Ticket URL: https://trac.osgeo.org/grass/ticket/1950#comment:2 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1945: wingrass: Command Line can't find grass.python libs
#1945: wingrass: Command Line can't find grass.python libs --+- Reporter: hamish| Owner: grass-dev@… Type: defect| Status: new Priority: critical | Milestone: 6.4.3 Component: Python| Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows XP Cpu: x86-32| --+- Comment(by hamish): Replying to [comment:5 glynn]: I don't think so. Unlike bash, Tcl doesn't re-parse the result of substitutions. ok Also the ... is not recognized as an internal or external command, operable program or batch file. error is generated by Windows in response to an attempt to execute a command; it wouldn't be generated by Tcl in response to a source command. It could be generated by the popen() call in G_gui_tcltk() (although the quoting looks okay), or by a Tcl exec command. I tried on linux with putting a space in $GISBASE, but it worked fine there. G_gui_tcltk() has a different #ifdef path for WIN32; I'm not sure if the tcltk code does too; I didn't see anything obvious. If I `set GRASS_DEBUG_GUI=true` to get the different tee+pipe behaviour from G_gui_tcltk(), it works and the module gui opens. Hamish -- Ticket URL: https://trac.osgeo.org/grass/ticket/1945#comment:6 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1949: module gui doesn't release if no map display window
#1949: module gui doesn't release if no map display window
-+--
Reporter: hamish | Owner: grass-dev@…
Type: defect | Status: new
Priority: normal | Milestone: 6.4.3
Component: wxGUI| Version: svn-develbranch6
Keywords: module guis |Platform: All
Cpu: All |
-+--
Comment(by annakrat):
Replying to [ticket:1949 hamish]:
When running a module in wxGUI (desktop icon startup) the control
buttons that are greyed out during the run never come back.
What exactly do you mean with control buttons?
test module: g.region from the menu, select -p flag from the Print tab
and [Run]
traceback error printed to the output tab:
{{{
Traceback (most recent call last):
display = self.parent.GetLayerTree().GetMapDisplay()
AttributeError
:
'NoneType' object has no attribute 'GetMapDisplay'
}}}
should be fixed in r55989.
--
Ticket URL: http://trac.osgeo.org/grass/ticket/1949#comment:1
GRASS GIS http://grass.osgeo.org
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1950: wingrass: replace rxvt with conhost.exe
#1950: wingrass: replace rxvt with conhost.exe ---+ Reporter: hamish | Owner: grass-dev@… Type: task | Status: new Priority: major | Milestone: 6.4.4 Component: Packaging | Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows 7 Cpu: All| ---+ Comment(by mmetz): Replying to [comment:2 hamish]: Replying to [ticket:1950 hamish]: IIUC, conhost.exe is new for Windows7, Windows XP doesn't have it. At least mine doesn't in C:\windows\system32. maybe it is csrss.exe there? Can you test if GRASS 7 with MSYS console works for you? Because there the MSYS console does not use rxvt and does not care about csrss.exe or conhost.exe, it simply starts sh as a shell in msys.bat. Markus M -- Ticket URL: http://trac.osgeo.org/grass/ticket/1950#comment:3 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1949: module gui doesn't release if no map display window
#1949: module gui doesn't release if no map display window -+-- Reporter: hamish | Owner: grass-dev@… Type: defect | Status: new Priority: normal | Milestone: 6.4.3 Component: wxGUI| Version: svn-develbranch6 Keywords: module guis |Platform: All Cpu: All | -+-- Comment(by hamish): Replying to [comment:1 annakrat]: should be fixed in r55989. Hi, it works now, nice. (please always commit to the dev branches first for testing, then backport once testing is confirmed :). otherwise it gets hard to untangle later since you can't trust the devbranch is newer, bulk backports erase previous fixes, last minute typos make it into the rel.br.., etc. all have happened in the past) thanks, Hamish -- Ticket URL: http://trac.osgeo.org/grass/ticket/1949#comment:2 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1949: module gui doesn't release if no map display window
#1949: module gui doesn't release if no map display window -+-- Reporter: hamish | Owner: grass-dev@… Type: defect | Status: new Priority: normal | Milestone: 6.4.3 Component: wxGUI| Version: svn-develbranch6 Keywords: module guis |Platform: All Cpu: All | -+-- Comment(by hamish): Replying to [comment:1 annakrat]: Replying to [ticket:1949 hamish]: When running a module in wxGUI (desktop icon startup) the control buttons that are greyed out during the run never come back. What exactly do you mean with control buttons? [Close] [Run] [Copy] [Help] at the bottom of the module gui window -- Ticket URL: http://trac.osgeo.org/grass/ticket/1949#comment:3 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1941: wxGUI fails with Japanese locale
#1941: wxGUI fails with Japanese locale --+- Reporter: venkat| Owner: grass-dev@… Type: defect| Status: new Priority: blocker | Milestone: 7.0.0 Component: wxGUI | Version: svn-trunk Keywords: wingrass |Platform: MSWindows 7 Cpu: x86-32| --+- Comment(by venkat): This issue seems to have been fixed in GRASS-6.4 but I think still remains in GRASS-7.0. -- Ticket URL: http://trac.osgeo.org/grass/ticket/1941#comment:3 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1891: wingrass: background dosbox from regular wxgui startup
#1891: wingrass: background dosbox from regular wxgui startup --+- Reporter: hamish| Owner: grass-dev@… Type: defect| Status: new Priority: major | Milestone: 6.4.3 Component: Default | Version: svn-releasebranch64 Keywords: wingrass |Platform: MSWindows 7 Cpu: x86-64| --+- Comment(by glynn): Replying to [comment:32 hamish]: rxvt is nothing to do with this bug beyond that's the only 6.x shell rxvt isn't any kind of shell. -- Ticket URL: http://trac.osgeo.org/grass/ticket/1891#comment:33 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1950: wingrass: replace rxvt with conhost.exe
#1950: wingrass: replace rxvt with conhost.exe ---+ Reporter: hamish | Owner: grass-dev@… Type: task | Status: new Priority: major | Milestone: 6.4.4 Component: Packaging | Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows 7 Cpu: All| ---+ Comment(by glynn): Replying to [ticket:1950 hamish]: the task: replace msys's rxvt.exe with Microsoft's built in conhost.exe terminal emulator. IIUC, conhost.exe is new for Windows7, Windows XP doesn't have it. At least mine doesn't in C:\windows\system32. If so that's a problem, half of the world's installed Windows systems are still running XP. The console window was part of the csrss.exe (Client-Server Runtime Sub- System) service prior to Windows 7. The exact details aren't important; all versions of Windows automatically create a console window if a console mode program is run without an existing console (e.g. from a desktop icon or start menu entry). (I've used it a bunch and not run into any problems, so beyond the above I'm not really seeing what all the fuss is about. what exactly about it is buggy?) I've found that copy+paste often caused rxvt to crash. But given that a number of 6.x modules either use curses or modify their behaviour depending upon whether isatty() returns true for stdin/stdout, the fact that rxvt won't work correctly with those modules can also be considered a bug. the bug where msys converts and C: to /c/ in strings when you don't want it to -- is that msys's rxvt or something else in their bash.exe? That's bash. And it's largely unavoidable if it's to be compatible with scripts written for Unix. -- Ticket URL: http://trac.osgeo.org/grass/ticket/1950#comment:5 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1950: wingrass: replace rxvt with conhost.exe
#1950: wingrass: replace rxvt with conhost.exe ---+ Reporter: hamish | Owner: grass-dev@… Type: task | Status: new Priority: major | Milestone: 6.4.4 Component: Packaging | Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows 7 Cpu: All| ---+ Comment(by glynn): Replying to [comment:4 hamish]: the existing trouble with grass6 cmd.exe Command Line mode is that shell scripts run within shell scripts can't find them. (e.g. v.in.lines(.sh) calling v.in.mapgen(.sh) seen in another bug report) That's because $GISBASE/scripts isn't added to PATH on Windows. bash has its own execution mechanism; in particular, it doesn't use PATHEXT, so you can't run a batch file without specifying the .bat or .cmd extension. But because shell scripts don't have a .sh extension (and the .sh extension probably isn't associated with bash anyhow), they can't just be executed normally. So GRASS 6.x creates a .bat file in $GISBASE/bin for each shell script. This works when executing scripts via cmd.exe, system(), subprocess.Popen(), etc, as those all use PATHEXT. But bash doesn't use PATHEXT, so it doesn't find the .bat files, and it doesn't find the scripts themselves because $GISBASE/scripts isn't in $PATH. Maybe adding $GISBASE/scripts to PATH in scripts/windows_launch.bat would work? Or it might not, as the updated PATH would be inherited by anything run from the script; presumably there's a reason for $GISBASE/scripts being excluded from PATH, rather than just being after $GISBASE/bin. -- Ticket URL: http://trac.osgeo.org/grass/ticket/1950#comment:6 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1950: wingrass: replace rxvt with conhost.exe
#1950: wingrass: replace rxvt with conhost.exe ---+ Reporter: hamish | Owner: grass-dev@… Type: task | Status: new Priority: major | Milestone: 6.4.4 Component: Packaging | Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows 7 Cpu: All| ---+ Comment(by mmetz): Replying to [comment:5 glynn]: Replying to [ticket:1950 hamish]: (I've used it a bunch and not run into any problems, so beyond the above I'm not really seeing what all the fuss is about. what exactly about it is buggy?) The only fuss I see is about keeping rxvt. I don't see what all the fuss is about. It has been settled for MSYS four years ago. I've found that copy+paste often caused rxvt to crash. But given that a number of 6.x modules either use curses or modify their behaviour depending upon whether isatty() returns true for stdin/stdout, the fact that rxvt won't work correctly with those modules can also be considered a bug. In rxvt, copy+paste never worked for me, but since I am sometimes trying to fix bugs for wingrass, I changed years ago to system console + sh in my local wingrass versions. It seems that using rxvt can be regarded as a bug. I updated msys.bat to the official stable version (MSYS 2009) in r55996,7 for rebr64, devbr6, respectively. About changes and testing: this change is not in a release candidate and far from being in an official release. A pity. -- Ticket URL: http://trac.osgeo.org/grass/ticket/1950#comment:7 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1950: wingrass: replace rxvt with conhost.exe
#1950: wingrass: replace rxvt with conhost.exe ---+ Reporter: hamish | Owner: grass-dev@… Type: task | Status: new Priority: major | Milestone: 6.4.4 Component: Packaging | Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows 7 Cpu: All| ---+ Comment(by hamish): Replying to [comment:7 mmetz]: The only fuss I see is about keeping rxvt. I don't care about keeping rxvt. I have no problem with replacing it if it's for the best. What I care about is last minute untested [in 6.x] changes in the release branch, and keeping the development team a nice community to participate in. In rxvt, copy+paste never worked for me, I've used it a lot and never had a problem with that, shrug. Just re- tested: in rxvt highlight + middle click + external ^V in textpad works perfectly for me (both directions), and a /lot/ less annoying than menu, mark, menu, cut, then menu, paste in a dosbox. fwiw rxvt font size options are nicer too.. ;-) but since I am sometimes trying to fix bugs for wingrass, I changed years ago to system console + sh in my local wingrass versions. so you've no experience with the current version and are basing your impression on some years old thing? It seems that using rxvt can be regarded as a bug. the question is if it is a serious enough problem to risk destabilizing the one wingrass command line option in 6.x which is 99% working properly, shortly before a public release. !@$!#@$ I updated msys.bat to the official stable version (MSYS 2009) in r55996,7 for rebr64, devbr6, respectively. thanks for respecting the just wait a bit requests of others... great collaborative working environment based on mutual respect we have here. by the way, you overwrote some local changes to the msys.bat script in your commits. (i.e. our %PART1% test to see that paths with spaces were correctly quoted, see original commit to svn + #905, #629) About changes and testing: this change is not in a release candidate and far from being in an official release. A pity. Perhaps, but 6.4.4 doesn't have to be that far away. Shoving core stuff in at the last minute is bad bad bad bad, no matter how innocent looking it seems. Now we have something else to spend many hours testing, which is annoying. --have you started that? was your local replacement+testing done for 6.x or trunk? I'm partly annoyed as we have a number of other major wingrass bugs to fix before release, it's hard to get anyone to work on them, and instead we seem to be wasting time and energy here discussing the bike shed. thanks, Hamish -- Ticket URL: https://trac.osgeo.org/grass/ticket/1950#comment:8 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
[GRASS-dev] (no subject)
Hi Eva, glad to see that we have the same research direction, and I'm very excited an expected to cooperate with you. On 04/24/2013 03:48 PM, anast...@centrum.sk wrote: Hi Corey, hi Ben, I am new at mailing list, but I deal with 3D kriging module for GRASS GIS as a PhD student for my dissertation thesis. The module enables to krige 2D and also 3D data (using 3D variogram), nowadays I continue testing and optimizing performance to make the module appropriate for publishing for other users. Best regards Eva Stopkova Bratislava, Slovak republic __**_ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/**mailman/listinfo/grass-devhttp://lists.osgeo.org/mailman/listinfo/grass-dev ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1727: v.in.lines not work on Window 7 64 bit
#1727: v.in.lines not work on Window 7 64 bit --+- Reporter: antonioall | Owner: grass-dev@… Type: defect | Status: reopened Priority: critical | Milestone: 6.4.3 Component: Vector | Version: svn-trunk Resolution: |Keywords: v.in.lines, g.tempfile Platform: MSWindows 7 | Cpu: x86-64 --+- Comment(by hamish): Glynn suggested the reason that the script-with-a-script was not working from the C:\ Command line version of wingrass 6.x was that the $GISBASE\scripts dir was missing from the %PATH% there. This is indeed the case, and adding it to the %PATH% allows v.in.lines (and any other script which calls another shell script) to run successfully. $GISBASE/bin is in there twice, that will need a test-before-adding too (maybe in env.bat?) tcl/tk eating the C: string remains open. Hamish -- Ticket URL: https://trac.osgeo.org/grass/ticket/1727#comment:18 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1946: wingrass: demolocation includes .svn/ cruft
#1946: wingrass: demolocation includes .svn/ cruft ---+ Reporter: hamish | Owner: grass-dev@… Type: defect | Status: new Priority: trivial| Milestone: 6.4.3 Component: Packaging | Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows XP Cpu: x86-32 | ---+ Comment(by hamish): attempt to remove the unpacked 'install_msruntime' dir in devbr6 r56001. -- Ticket URL: https://trac.osgeo.org/grass/ticket/1946#comment:13 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1947: wingrass: %APPDATA%/GRASS6/grassrc6 and rc files
#1947: wingrass: %APPDATA%/GRASS6/grassrc6 and rc files ---+ Reporter: hamish | Owner: grass-dev@… Type: defect | Status: new Priority: normal | Milestone: 6.4.3 Component: Packaging | Version: svn-develbranch6 Keywords: wingrass |Platform: MSWindows XP Cpu: x86-32 | ---+ Comment(by hamish): committed for testing in devbr6 with r56001,2. see also earlier discussion: http://thread.gmane.org/gmane.comp.gis.grass.devel/46154/ Hamish -- Ticket URL: https://trac.osgeo.org/grass/ticket/1947#comment:1 GRASS GIS http://grass.osgeo.org ___ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1946: wingrass: demolocation includes .svn/ cruft
#1946: wingrass: demolocation includes .svn/ cruft
---+
Reporter: hamish | Owner: grass-dev@…
Type: defect | Status: new
Priority: trivial| Milestone: 6.4.3
Component: Packaging | Version: svn-develbranch6
Keywords: wingrass |Platform: MSWindows XP
Cpu: x86-32 |
---+
Comment(by hamish):
Replying to [comment:12 glynn]:
Replying to [ticket:1946 hamish]:
the nightly build wingrass installers ship '.svn/' files for the
demolocation.
I think that this isn't restricted to Windows; the builddemolocation
target in the top-level Makefile simply copies the demolocation
directory
to dist.arch (using tar c ... | tar x ...).
that is correct, it's in all 'make install' for svn builds.
In 7.x, the demolocation directory has a Makefile which explicitly
creates target directories and copies files with $(INSTALL_DATA).
since there's no `Other.make` in grass6 I'd just go with the cheap dirty
approach-of-least-change:
{{{
Index: Makefile
===
--- Makefile(revision 55903)
+++ Makefile(working copy)
@@ -389,6 +389,13 @@
builddemolocation:
test -d ${ARCH_DISTDIR} || ${MAKE_DIR_CMD} ${ARCH_DISTDIR}
-tar cBf - demolocation | (cd ${ARCH_DISTDIR}/ ; tar xBfo - )
2/dev/null
+ -(cd ${ARCH_DISTDIR}/demolocation ; \
+ rm -rf .svn ; \
+ rm -rf PERMANENT/.svn ; \
+ rm -rf PERMANENT/vector/.svn ; \
+ rm -rf PERMANENT/vector/mysites/.svn ; \
+ rm -rf PERMANENT/vector/point/.svn ; \
+ rm -rf PERMANENT/dbf/.svn )
@ echo GISDBASE: ${RUN_GISBASE} ${RUN_GISRC}
@ echo LOCATION_NAME: demolocation ${RUN_GISRC}
@ echo MAPSET: PERMANENT ${RUN_GISRC}
}}}
I notice that the last part creates a file called demolocation/.grassrc65,
when it should perhaps be .grassrc6? That's named by Platform.make.in:
{{{
RUN_GISRC =
${ARCH_DISTDIR}/demolocation/.grassrc${GRASS_VERSION_MAJOR}${GRASS_VERSION_MINOR}
}}}
(RUN_GISRC is used by e.g. Man.make at build time, with the GISRC=
override set to the same variable, so it doesn't matter much what it's
called)
?,
Hamish
--
Ticket URL: https://trac.osgeo.org/grass/ticket/1946#comment:14
GRASS GIS http://grass.osgeo.org
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev
Re: [GRASS-dev] [GRASS GIS] #1727: v.in.lines not work on Window 7 64 bit
#1727: v.in.lines not work on Window 7 64 bit
--+-
Reporter: antonioall | Owner: grass-dev@…
Type: defect | Status: reopened
Priority: critical | Milestone: 6.4.3
Component: Vector | Version: svn-trunk
Resolution: |Keywords: v.in.lines, g.tempfile
Platform: MSWindows 7 | Cpu: x86-64
--+-
Comment(by hamish):
re. script-called-from-a-script failure from the 6.x wingrass Command Line
and GIS.m, here's a patch.
{{{
Index: lib/init/init.bat
===
--- lib/init/init.bat (revision 56002)
+++ lib/init/init.bat (working copy)
@@ -24,8 +24,10 @@
set SAVEPATH=%PATH%
rem
-rem DON'T include the scripts/ directory in PATH -- The .bat files in the
-rem bin/ directory are what execute the scripts on Windows.
+rem The .bat files in the \bin directory are what execute the scripts
+rem on Windows. The \scripts dir is added to the PATH automatically by
+rem the wxGUI; for Command Line mode and GIS.m, shell scripts called
from
+rem other shell scripts need to have %GISBASE%\scripts in the PATH.
rem
if exist %APPDATA%\GRASS6\env.bat (
@@ -121,9 +123,12 @@
set PYTHONPATH=%PYTHONPATH%;%GISBASE%\etc\python
)
-if %GRASS_GUI%==text goto text
if %GRASS_GUI%==wxpython goto wxpython
+PATH=%PATH%;%GISBASE%\scripts
+if %GRASS_GUI%==text goto text
+
+rem Tcl/Tk GUI setup
if not %GRASS_WISH%== (
%GRASS_WISH% %GISBASE%\etc\gis_set.tcl
) else (
}}}
any comments before committing it?
Hamish
--
Ticket URL: https://trac.osgeo.org/grass/ticket/1727#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
