[Home]

Summary:ASTERISK-17125: [patch] Sqlite3 requires -lpthread to build in configure.ac
Reporter:Philip Prindeville (pprindeville)Labels:
Date Opened:2010-12-16 23:10:44.000-0600Date Closed:2010-12-17 15:06:14.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/BuildSystem
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20101217__issue18493.diff.txt
( 1) asterisk-1.8-sqlite3.patch
Description:At least on astlinux, our libsqlite3.so requires linkage with -lpthread.

Alas, the:

AST_EXT_LIB_CHECK([SQLITE3], ...)

requires [${LIB_PTHREADS}] same as [ICAL] does.


****** STEPS TO REPRODUCE ******

Build under astlinux trunk against version 1.8.
Comments:By: Tilghman Lesher (tilghman) 2010-12-17 13:55:49.000-0600

What is the output of:

pkg-config --libs sqlite3
By: Philip Prindeville (pprindeville) 2010-12-17 14:30:21.000-0600

$ cat build_i586/staging_dir/usr/lib/pkgconfig/sqlite3.pc
# Package Information for pkg-config

prefix=/usr
exec_prefix=${prefix}
libdir=${exec_prefix}/lib
includedir=${prefix}/include

Name: SQLite
Description: SQL database engine
Version: 3.6.23.1
Libs: -L${libdir} -lsqlite3
Libs.private: -lpthread
Cflags: -I${includedir}
$ pkg-config --libs sqlite3
-lsqlite3  
$

But then, we're not setting PKG_CONFIG_PATH or PKG_CONFIG_LIBDIR, either.
By: Tilghman Lesher (tilghman) 2010-12-17 14:31:17.000-0600

Note that you'll need to run './bootstrap.sh' after this patch to regenerate the configure script.
By: Digium Subversion (svnbot) 2010-12-17 15:03:07.000-0600

Repository: asterisk
Revision: 298817

U   branches/1.6.2/configure
U   branches/1.6.2/configure.ac

------------------------------------------------------------------------
r298817 | tilghman | 2010-12-17 15:03:07 -0600 (Fri, 17 Dec 2010) | 8 lines

Also include PTHREAD_LIBS and PTHREAD_CFLAGS for SQLite 3, as it's needed on some platforms.

(closes issue ASTERISK-17125)
Reported by: pprindeville
Patches:
      asterisk-1.8-sqlite3.patch uploaded by pprindeville (license 347)
Tested by: pprindeville

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

http://svn.digium.com/view/asterisk?view=rev&revision=298817
By: Digium Subversion (svnbot) 2010-12-17 15:04:22.000-0600

Repository: asterisk
Revision: 298818

_U  branches/1.8/
U   branches/1.8/configure
U   branches/1.8/configure.ac

------------------------------------------------------------------------
r298818 | tilghman | 2010-12-17 15:04:22 -0600 (Fri, 17 Dec 2010) | 15 lines

Merged revisions 298817 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........
 r298817 | tilghman | 2010-12-17 15:03:06 -0600 (Fri, 17 Dec 2010) | 8 lines
 
 Also include PTHREAD_LIBS and PTHREAD_CFLAGS for SQLite 3, as it's needed on some platforms.
 
 (closes issue ASTERISK-17125)
  Reported by: pprindeville
  Patches:
        asterisk-1.8-sqlite3.patch uploaded by pprindeville (license 347)
  Tested by: pprindeville
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=298818
By: Digium Subversion (svnbot) 2010-12-17 15:06:14.000-0600

Repository: asterisk
Revision: 298819

_U  trunk/
U   trunk/configure
U   trunk/configure.ac

------------------------------------------------------------------------
r298819 | tilghman | 2010-12-17 15:06:14 -0600 (Fri, 17 Dec 2010) | 22 lines

Merged revisions 298818 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
 r298818 | tilghman | 2010-12-17 15:04:21 -0600 (Fri, 17 Dec 2010) | 15 lines
 
 Merged revisions 298817 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.6.2
 
 ........
   r298817 | tilghman | 2010-12-17 15:03:06 -0600 (Fri, 17 Dec 2010) | 8 lines
   
   Also include PTHREAD_LIBS and PTHREAD_CFLAGS for SQLite 3, as it's needed on some platforms.
   
   (closes issue ASTERISK-17125)
    Reported by: pprindeville
    Patches:
          asterisk-1.8-sqlite3.patch uploaded by pprindeville (license 347)
    Tested by: pprindeville
 ........
................

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

http://svn.digium.com/view/asterisk?view=rev&revision=298819