[GRASS-dev] trac #637: spaces in path names (wx + tcl)

GRASS GRASS developer
1

Hi,

re. stalled bug #637,
  https://trac.osgeo.org/grass/ticket/637#comment:11

I take it the wxGUI use of 'v.db.connect -g' still should explicitly
set the fs='|' and split() on that instead of space?

Also Tcl/Tk versions of same still need attention / testing /
backporting:

https://trac.osgeo.org/grass/log/grass/branches/develbranch_6/gui/tcltk/gis.m

https://trac.osgeo.org/grass/changeset/38295
https://trac.osgeo.org/grass/changeset/38296
https://trac.osgeo.org/grass/changeset/38426
https://trac.osgeo.org/grass/changeset/38427

and in general C code,
https://trac.osgeo.org/grass/changeset/38436

... maybe others

thanks,
Hamish

2

Hamish wrote:

I take it the wxGUI use of 'v.db.connect -g' still should explicitly
set the fs='|' and split() on that instead of space?

Yes.

Also Tcl/Tk versions of same still need attention / testing /
backporting:

https://trac.osgeo.org/grass/log/grass/branches/develbranch_6/gui/tcltk/gis.m

https://trac.osgeo.org/grass/changeset/38295
https://trac.osgeo.org/grass/changeset/38296
https://trac.osgeo.org/grass/changeset/38426
https://trac.osgeo.org/grass/changeset/38427

and in general C code,
https://trac.osgeo.org/grass/changeset/38436

The dbmscap fix is in the wrong place. Any quoting should be done
within db_start_driver() (and the Unix version shouldn't be using the
shell; there's no need for it).

More generally, most of those quoting fixes are bogus. The solution is
to use G_spawn() rather than using the shell. Using the shell is a
bug; quoting is just a workaround.

--
Glynn Clements <glynn@gclements.plus.com>