Opened 6 years ago

Closed 6 years ago

#3444 closed defect (fixed)

v.concave.hull -script type definition for table column not compatible with PostgreSQL

Reported by: harrik Owned by: grass-dev@…
Priority: normal Milestone:
Component: Addons Version: 7.2.0
Keywords: v.concave.hull, PostgreSQL Cc:
CPU: Unspecified Platform: Unspecified

Description

Very simple. The GRASS Addon 'v.concave.hull' has a column definition that does not work with the PostgreSQL backend:

Line 98:
grass.run_command('v.db.addtable', map = out_lines,
                   col = 'cat integer,length double', quiet = True)

That gives an error, whereas the simple replacement of length double with length double precision makes the script run:

grass.run_command('v.db.addtable', map = out_lines,
                      col = 'cat integer,length double precision', quiet = True)

This should run at least with the sql-based backends since based on MySQL and SQLite documentation the double precision is the full type name anyway, but I do not know about DBF. But from what I've understood, the use of DBF is not recommended any more anyway.

I could easily fix this myself, but I do not have write access to sources.

Harri

Change History (3)

comment:1 by mlennert, 6 years ago

Fixed in r71741. Please reinstall the extension and test.

comment:2 by harrik, 6 years ago

Works now.

comment:3 by martinl, 6 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.