Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#2044 closed enhancement (fixed)

NC demolocation should include sqlite dirs in mapsets

Reported by: mlennert Owned by: grass-dev@…
Priority: normal Milestone: 7.0.0
Component: Database Version: svn-trunk
Keywords: sqlite Cc:
CPU: Unspecified Platform: Unspecified

Description

As mention in #1358, the sqlite driver does not work by default in grass7 if there is no sqlite directory in the mapset. When the user creates a new mapset, such a directory is created automatically. However, the NC demo data for GRASS7 at [1] does not contain such directories in the user1 and the landsat mapsets.

While it would probably be preferrable to alter the sqlite driver so that it creates the directory if needed, it might be worthwile, in the meantime, to include these directories to avoid bad first time experiences for new users.

[1] http://grass.osgeo.org/sampledata/north_carolina/nc_spm_08_grass7.tar.gz

Change History (5)

comment:1 Changed 6 years ago by neteler

For now I have fixed the package (also the reduced one) on the server.

Still the software update is needed, hence not closing.

comment:2 in reply to:  1 Changed 6 years ago by neteler

Component: DefaultDatabase
Keywords: sqlite added

Replying to neteler:

For now I have fixed the package (also the reduced one) on the server.

Still the software update is needed, hence not closing.

To be continued in ticket #1358, closing here since data set issue is solved.

comment:3 Changed 6 years ago by neteler

Resolution: fixed
Status: newclosed

comment:4 Changed 6 years ago by hamish

When the user creates a new mapset, such a directory is created automatically.

that should happen at happen whenever the directory is needed and it doesn't already exist. Really, it should not be created when the mapset is created, only on demand. (flush out the bugs now rather than later, see earlier discussions about dbf/ ...)

While it would probably be preferrable to alter the sqlite driver so that it creates the directory if needed,

it's not just preferable, it's essential, especially for people changing versions and so entering old mapsets with new versions of the software. But anyway, hopefully that problem was fixed in trunk some months ago with r57276. AFAICT that has not been backported to GRASS 6, if it would be needed there.

To be continued in ticket #1358

regards, Hamish

comment:5 in reply to:  4 Changed 6 years ago by neteler

Replying to hamish:

AFAICT that has not been backported to GRASS 6, if it would be needed there.

Why should it be backported? The sqlite.db is not residing in a subdirectory in G6.

Note: See TracTickets for help on using tickets.