[GRASS-dev] [GRASS GIS] #1862: Corrections GRASS 7.0 help files db.connect and v.db.connect

#1862: Corrections GRASS 7.0 help files db.connect and v.db.connect
-------------------------+--------------------------------------------------
Reporter: pvanbosgeo | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 7.0.0
Component: Default | Version: svn-trunk
Keywords: | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
In db.connect, it is suggested the default driver is dbf
For both db.connect and v.db.connect the default database name is said to
be $GISDBASE/$LOCATION_NAME/$MAPSET/dbf/

This is obviously not a defect, but there does not seem to be a 'ticket
type' that fits the bill in this case. Perhaps one can added for reports
about issues in the help files / documentation?

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

#1862: Corrections GRASS 7.0 help files db.connect and v.db.connect
-------------------------+--------------------------------------------------
Reporter: pvanbosgeo | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 7.0.0
Component: Docs | Version: svn-trunk
Keywords: | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by neteler):

  * component: Default => Docs

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

#1862: Corrections GRASS 7.0 help files db.connect and v.db.connect
-------------------------+--------------------------------------------------
Reporter: pvanbosgeo | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 7.0.0
Component: Docs | Version: svn-trunk
Keywords: | Platform: Unspecified
      Cpu: Unspecified |
-------------------------+--------------------------------------------------

Comment(by neteler):

The "docs" tag is there. The following manual pages may need an update to
SQLite:

  * db.columns/db.columns.html
  * db.connect/db.connect.html
  * db.copy/db.copy.html
  * db.describe/db.describe.html
  * db.tables/db.tables.html

Patches appreciated :slight_smile:

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

Ah, OK, had overlooked that component tag, that is what happens when trying to file a ticket quickly in between other tasks :-/ . Anyway, good to know for next time. I guess even patches for the manual pages require some serious programming background… otherwise I would have been happy to provide them.

···

On Thu, Jan 24, 2013 at 9:55 PM, GRASS GIS <trac@osgeo.org> wrote:

#1862: Corrections GRASS 7.0 help files db.connect and v.db.connect
-------------------------±-------------------------------------------------
Reporter: pvanbosgeo | Owner: grass-dev@…
Type: defect | Status: new
Priority: normal | Milestone: 7.0.0

Component: Docs | Version: svn-trunk

Keywords: | Platform: Unspecified
Cpu: Unspecified |
-------------------------±-------------------------------------------------

Comment(by neteler):

The “docs” tag is there. The following manual pages may need an update to
SQLite:

  • db.columns/db.columns.html
  • db.connect/db.connect.html
  • db.copy/db.copy.html
  • db.describe/db.describe.html
  • db.tables/db.tables.html

Patches appreciated :slight_smile:


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

Hi Paulo,

not at all. You may post the change suggestions even as email and
we'll merge them in manually, not too much work in this case.
I just don't manage to write the better examples myself in this period.

Thanks,
Markus

Hi Markus,

OK, as an example, in db.connect, the ‘offending’ text is under the ‘Parameter’ heading. The default driver name and the default database name should change (marked with >>):

On Fri, Jan 25, 2013 at 9:31 AM, Paulo van Breugel
<p.vanbreugel@gmail.com> wrote:

Hi Markus,

OK, as an example, in db.connect, the 'offending' text is under the
'Parameter' heading. The default driver name and the default database name
should change (marked with >>):

-----------------

driver=name

   Name of database driver

   Options: pg, odbc, mysql, ogr, dbf, sqlite

   >> Default: sqlite

database=name

   Name of database

   >> Default: $GISDBASE/$LOCATION_NAME/$MAPSET/sqlite/sqlite.db

schema=name

   Database schema

   Do not use this option if schemas are not supported by driver/database
server

group=string

   Default group of database users to which select privilege is granted

--------------------

Just for my understanding, these parts of the manual page are not in the
html file
(http://svn.osgeo.org/grass/grass/trunk/db/db.connect/db.connect.html) and
I
assume they are generated / parsed from the actual code? With no
understanding of the source code (I had a look out of curiosity, but that
was an exercise in futility), I was wondering how/if I could suggest
changes
that would actually be more helpful then just mentioning them like above.

I think the demolocation in the source tree needs to be changed to
sqlite, because db.connect and v.db.connect use
db_get_default_*_name().

Markus M

Cheers,

Paulo

On Fri 25 Jan 2013 07:09:56 AM CET, Markus Neteler wrote:

Hi Paulo,

not at all. You may post the change suggestions even as email and
we'll merge them in manually, not too much work in this case.
I just don't manage to write the better examples myself in this period.

Thanks,
Markus

_______________________________________________
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

On Fri, Jan 25, 2013 at 9:50 AM, Markus Metz
<markus.metz.giswork@gmail.com> wrote:

On Fri, Jan 25, 2013 at 9:31 AM, Paulo van Breugel
<p.vanbreugel@gmail.com> wrote:

Hi Markus,

OK, as an example, in db.connect, the 'offending' text is under the
'Parameter' heading. The default driver name and the default database name
should change (marked with >>):

-----------------

driver=name

   Name of database driver

   Options: pg, odbc, mysql, ogr, dbf, sqlite

   >> Default: sqlite

database=name

   Name of database

   >> Default: $GISDBASE/$LOCATION_NAME/$MAPSET/sqlite/sqlite.db

schema=name

   Database schema

   Do not use this option if schemas are not supported by driver/database
server

group=string

   Default group of database users to which select privilege is granted

--------------------

Just for my understanding, these parts of the manual page are not in the
html file
(http://svn.osgeo.org/grass/grass/trunk/db/db.connect/db.connect.html) and
I
assume they are generated / parsed from the actual code? With no
understanding of the source code (I had a look out of curiosity, but that
was an exercise in futility), I was wondering how/if I could suggest
changes
that would actually be more helpful then just mentioning them like above.

I think the demolocation in the source tree needs to be changed to
sqlite, because db.connect and v.db.connect use
db_get_default_*_name().

The demolocation adjusted for trunk in r54771. The manuals will be
updated after make distclean; configure; make.

Markus M

Cheers,

Paulo

On Fri 25 Jan 2013 07:09:56 AM CET, Markus Neteler wrote:

Hi Paulo,

not at all. You may post the change suggestions even as email and
we'll merge them in manually, not too much work in this case.
I just don't manage to write the better examples myself in this period.

Thanks,
Markus

_______________________________________________
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

#1862: Corrections GRASS 7.0 help files db.connect and v.db.connect
-------------------------+--------------------------------------------------
Reporter: pvanbosgeo | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 7.0.0
Component: Docs | Version: svn-trunk
Keywords: | Platform: All
      Cpu: Unspecified |
-------------------------+--------------------------------------------------
Changes (by neteler):

  * platform: Unspecified => All

Comment:

db.connect improved in r55009

db.copy improved in r55011

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

#1862: Corrections GRASS 7.0 help files db.connect and v.db.connect
-------------------------+--------------------------------------------------
  Reporter: pvanbosgeo | Owner: grass-dev@…
      Type: defect | Status: closed
  Priority: normal | Milestone: 7.0.0
Component: Docs | Version: svn-trunk
Resolution: fixed | Keywords:
  Platform: All | Cpu: Unspecified
-------------------------+--------------------------------------------------
Changes (by neteler):

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

Comment:

Completed in r59358 and r59359. Closing.

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