2015-11-12 20:25 GMT+01:00 Lonnie Abelbeck:
> My current solution is to apply this build patch:
....
> And that seems to restore the previous behavior to dynamically link the 
> sqlite3 shell tool.

Actually, Makefile.in is generated from Makefile.am, so the correct patch would
be as below in the SQLite sources. I noticed the same for the Cygwin build
(I'm Cygwin's SQLite package maintainer), and put the patch below in the
build system, so it will be applied with every new SQLite release. Problem
solved for me.

> It would be nice if like Fossil (--with-internal-sqlite), sqlite had a 
> configure option to determine if the shell tool would be statically or 
> dynamically linked, ex. --with-static-shell-tool
>
> I would expect sqlite would default to dynamically linked (if --enable-shared 
> is passed), not sure why the change to static.
+1
If it would be configurable, that indeed would be best. Then it even
doesn't matter to me
what the default is.

My guess is the more distributions (e.g. Fedora, Ubuntu ....) will
make (or have already
made) the same modification, since it's much easier to apply a patch than change
a distribution policy ;-)

Regards,
       Jan Nijtmans

==============================
--- autoconf/Makefile.am
+++ autoconf/Makefile.am
@@ -4,16 +4,16 @@
 lib_LTLIBRARIES = libsqlite3.la
 libsqlite3_la_SOURCES = sqlite3.c
 libsqlite3_la_LDFLAGS = -no-undefined -version-info 8:6:8

 bin_PROGRAMS = sqlite3
-sqlite3_SOURCES = shell.c sqlite3.c sqlite3.h
-sqlite3_LDADD = @READLINE_LIBS@
-sqlite3_CFLAGS = $(AM_CFLAGS)
+sqlite3_SOURCES = shell.c
+sqlite3_LDADD = $(top_builddir)/libsqlite3.la @READLINE_LIBS@
+sqlite3_DEPENDENCIES = $(top_builddir)/libsqlite3.la

 include_HEADERS = sqlite3.h sqlite3ext.h

 EXTRA_DIST = sqlite3.1 tea
 pkgconfigdir = ${libdir}/pkgconfig
 pkgconfig_DATA = sqlite3.pc

 man_MANS = sqlite3.1

Reply via email to