[GRASS-dev] Enable compilation with SQLite by deafut

GRASS GIS GRASS GIS developer
1

Hi,

I just noticed that the default value in ./configure for SQLite support is “no”. Since sqlite is the default for GRASS 7 wouldn’t make sense to have it enabled by default?

For example, wxGUI does not even considers missing SQLite as a possibility. I’m not sure if it is right and if modules are more robust. However, to make compilation easier for (compilation) first time users, I suggest to change the default to “yes”.

I don’t know how to do it, configure(.in) is still mysterious for me.

Thanks,

Vaclav

./configure --help | grep sql

–with-sqlite support SQLite functionality (default: no)

2

On Thu, Jun 26, 2014 at 10:06 PM, Vaclav Petras <wenzeslaus@gmail.com> wrote:

Hi,

I just noticed that the default value in ./configure for SQLite support is
"no". Since sqlite is the default for GRASS 7 wouldn't make sense to have it
enabled by default?

For example, wxGUI does not even considers missing SQLite as a possibility.
I'm not sure if it is right and if modules are more robust. However, to make
compilation easier for (compilation) first time users, I suggest to change
the default to "yes".

To me that sounds reasonable.

I don't know how to do it, configure(.in) is still mysterious for me.

Very easy:

- in "configure.in" line 253, take out the ", no"
- then run autoconf-2.13
- run configure ... to test

Markus

3

On Thu, Jun 26, 2014 at 4:25 PM, Markus Neteler <neteler@osgeo.org> wrote:

On Thu, Jun 26, 2014 at 10:06 PM, Vaclav Petras <wenzeslaus@gmail.com>
wrote:
> Hi,
>
> I just noticed that the default value in ./configure for SQLite support
is
> "no". Since sqlite is the default for GRASS 7 wouldn't make sense to
have it
> enabled by default?
>
> For example, wxGUI does not even considers missing SQLite as a
possibility.
> I'm not sure if it is right and if modules are more robust. However, to
make
> compilation easier for (compilation) first time users, I suggest to
change
> the default to "yes".

To me that sounds reasonable.

OK, done in r60995

> I don't know how to do it, configure(.in) is still mysterious for me.

Very easy:

- in "configure.in" line 253, take out the ", no"
- then run autoconf-2.13
- run configure ... to test

Thanks, this was really easy, but my understanding is still low.

After autoconf2.13, I tested with make distclean, configure and make.

Vaclav

http://trac.osgeo.org/grass/changeset/60995

Markus