[GRASS-dev] Re: [GRASS GIS] #902: nviz (tcl) fails on wingrass

[GRASS GIS]

#902: nviz (tcl) fails on wingrass
-----------------------+----------------------------------------------------
  Reporter:  hamish    |       Owner:  grass-dev at lists.osgeo.org
      Type:  defect    |      Status:  new                      
  Priority:  critical  |   Milestone:  6.4.0                    
 Component:  NVIZ      |     Version:  svn-trunk                
Resolution:            |    Keywords:                           
  Platform:  All       |         Cpu:  All                      
-----------------------+----------------------------------------------------
Changes (by glynn):

  * keywords:  wingrass =>
  * platform:  MSWindows XP => All
  * version:  svn-releasebranch64 => svn-trunk
  * cpu:  x86-32 => All

Comment:

 Replying to [ticket:902 hamish]:

 > with the latest WinGrass nightly build installer (r40766), installed in
 C:\Program Files\, nviz (tcl) fails to start.
 >
 > even from the MSys command prompt there error is:
 >
 {{{
 GRASS 6.4> nviz --help
 Sorry <Files/GRASS-64-SNV/etc/nviz2.2/script/nviz2.2_script> is not a
 valid option
 }}}

 This is due to Ngetargs() in nviz_init.c.

 It tries to reconstruct the command line with:
 {{{
         sprintf(tmp2, "%s %s %s", cmd, argv0, tmp);
 }}}
 then convert it back to a list with Tcl_SplitList().

 This could be hacked around by using braces, but it should probably be
 using Tcl_ObjGetVar2(), Tcl_NewListObj() and Tcl_ListObjAppendList(). Or
 simpler still, change Ngetargs to return `argv[0]` separately (and just
 ignore `argv[1]`), and change parse_command() to match.

 Oh, and it's broken in all versions for all platforms (spaces in pathnames
 are uncommon on Unix, but not prohibited).

-- 
Ticket URL: <https://trac.osgeo.org/grass/ticket/902#comment:2>
GRASS GIS <http://grass.osgeo.org>