[GRASS-dev] [GRASS GIS] #1245: v.db.connect parsing

#1245: v.db.connect parsing
-----------------------------------+----------------------------------------
Reporter: hamish | Owner: grass-dev@…
     Type: defect | Status: new
Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Keywords: v.db.connect, scripts | Platform: All
      Cpu: All |
-----------------------------------+----------------------------------------
Hi,

just a placeholder bug report for the records; as reported on the mailing
list.

sqlite+v.in.ogr adds a table name to the table number reported by
v.db.connect -p,-g as "number/name". So if parsing for the table number we
need to use the -l flag as well to suppress the name.

the following shell scripts are affected, need to check the python
scripts/lib for this too.

v.colors (fixed)
d.vect.thematic
v.db.addcol
v.db.addtable
v.db.dropcol
v.db.droptable
v.db.join
v.db.renamecol
v.db.update
v.dissolve
v.out.gpsbabel
v.rast.stats
v.report

Hamish

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

#1245: v.db.connect parsing
-----------------------------------+----------------------------------------
Reporter: hamish | Owner: hamish
     Type: defect | Status: assigned
Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Keywords: v.db.connect, scripts | Platform: All
      Cpu: All |
-----------------------------------+----------------------------------------
Changes (by hamish):

* cc: grass-dev@… (added)
  * owner: grass-dev@… => hamish
  * status: new => assigned

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

#1245: v.db.connect parsing
-----------------------------------+----------------------------------------
Reporter: hamish | Owner: hamish
     Type: defect | Status: assigned
Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Keywords: v.db.connect, scripts | Platform: All
      Cpu: All |
-----------------------------------+----------------------------------------
Description changed by hamish:

Old description:

Hi,

just a placeholder bug report for the records; as reported on the mailing
list.

sqlite+v.in.ogr adds a table name to the table number reported by
v.db.connect -p,-g as "number/name". So if parsing for the table number
we need to use the -l flag as well to suppress the name.

the following shell scripts are affected, need to check the python
scripts/lib for this too.

v.colors (fixed)
d.vect.thematic
v.db.addcol
v.db.addtable
v.db.dropcol
v.db.droptable
v.db.join
v.db.renamecol
v.db.update
v.dissolve
v.out.gpsbabel
v.rast.stats
v.report

Hamish

New description:

Hi,

just a placeholder bug report for the records; as reported on the mailing
list.

sqlite+v.in.ogr adds a table name to the table number reported by
v.db.connect -p,-g as "number/name". So if parsing for the table number we
need to use the -l flag as well to suppress the name.

the following shell scripts are affected, need to check the python
scripts/lib for this too.

{{{
v.colors (fixed)
d.vect.thematic
v.db.addcol
v.db.addtable
v.db.dropcol
v.db.droptable
v.db.join
v.db.renamecol
v.db.update
v.dissolve
v.out.gpsbabel
v.rast.stats
v.report
}}}

Hamish

--

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

#1245: v.db.connect parsing
-----------------------------------+----------------------------------------
Reporter: hamish | Owner: hamish
     Type: defect | Status: assigned
Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Keywords: v.db.connect, scripts | Platform: All
      Cpu: All |
-----------------------------------+----------------------------------------

Comment(by hamish):

python scripts to check:
{{{
lib/python/vector.py

v.db.addtable
v.db.droptable
v.db.reconnect.all
}}}

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

#1245: v.db.connect parsing
-----------------------------------+----------------------------------------
Reporter: hamish | Owner: hamish
     Type: defect | Status: assigned
Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Keywords: v.db.connect, scripts | Platform: All
      Cpu: All |
-----------------------------------+----------------------------------------

Comment(by hamish):

all the python in trunk is ok

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

#1245: v.db.connect parsing
-----------------------------------+----------------------------------------
Reporter: hamish | Owner: hamish
     Type: defect | Status: assigned
Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Keywords: v.db.connect, scripts | Platform: All
      Cpu: All |
-----------------------------------+----------------------------------------

Comment(by hamish):

6.5svn scripts updated in r44641.

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

#1245: v.db.connect parsing
-----------------------------------+----------------------------------------
Reporter: hamish | Owner: hamish
     Type: defect | Status: assigned
Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Keywords: v.db.connect, scripts | Platform: All
      Cpu: All |
-----------------------------------+----------------------------------------

Comment(by neteler):

Hamish, what about the 6.4 backport?

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

#1245: v.db.connect parsing
-----------------------------------+----------------------------------------
Reporter: hamish | Owner: hamish
     Type: defect | Status: assigned
Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Keywords: v.db.connect, scripts | Platform: All
      Cpu: All |
-----------------------------------+----------------------------------------

Comment(by hamish):

Replying to [comment:6 neteler]:
> Hamish, what about the 6.4 backport?

as it was not a simple incisive change, before backporting I thought to
leave it in 6.5svn for a while in case there were any typos.

aka I haven't tested the changes to see if they still work.
other than that, no reason not to backport. (not being lazy with
backports; I try not to backport anything I haven't tested. so just being
lazy with testing :slight_smile:

Hamish

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

#1245: v.db.connect parsing
---------------------+------------------------------------------------------
  Reporter: hamish | Owner: hamish
      Type: defect | Status: closed
  Priority: major | Milestone: 6.4.1
Component: Vector | Version: svn-releasebranch64
Resolution: fixed | Keywords: v.db.connect, scripts
  Platform: All | Cpu: All
---------------------+------------------------------------------------------
Changes (by hamish):

  * status: assigned => closed
  * resolution: => fixed

Comment:

backported to relbr64 in r45260.

possibly broken (poorly tested) is less bad than known broken (sqlite ops
in 6.4.0).

Hamish

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