Summary: | [Gtk] Use the SQLite3 headers from WebKitLibraries if sqlite3 is undetected | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jan Alonzo <jmalonzo> | ||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | aroben, eric, zecke | ||||
Priority: | P2 | Keywords: | Gtk | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Jan Alonzo
2009-09-19 00:38:10 PDT
Created attachment 39816 [details]
Patch v1
Comment on attachment 39816 [details] Patch v1 >+ PKG_CHECK_MODULES([SQLITE3], >+ [sqlite3 >= $SQLITE_REQUIRED_VERSION], >+ [sqlite3_has_pkg_config=yes], >+ [sqlite3_has_pkg_config=no]) >+ if test "$sqlite3_has_pkg_config" = "no"; then >+ AC_SEARCH_LIBS([sqlite3_open16], [sqlite3], >+ [SQLITE3_LIBS="$LIBS";SQLITE3_CFLAGS="-I $srcdir/WebKitLibraries/WebCoreSQLite3"], >+ [AC_MSG_ERROR([SQLite3 is required to enable Database support])]) >+ fi Silly question, shouldn't you first try to use the headers that might come with the library you just found with AC_SEARCH_LIBS? Or are you assuming that if there's no sqlite pkg-config file the only library you'll find is the one in WebKitLibraries? > AC_SUBST([SQLITE3_CFLAGS]) > AC_SUBST([SQLITE3_LIBS]) > fi Looks fine to me, although I'm not sure you really want to be depending on those headers. Xan seems to have an objection though, so waiting for you to respond to that before marking r+. Or rather, Xan can mark it r+ if he agrees. Did we ever get a report that someone wanted to use sqlite3 but didn't have pkg-config files? (In reply to comment #2) > (From update of attachment 39816 [details]) > >+ PKG_CHECK_MODULES([SQLITE3], > >+ [sqlite3 >= $SQLITE_REQUIRED_VERSION], > >+ [sqlite3_has_pkg_config=yes], > >+ [sqlite3_has_pkg_config=no]) > >+ if test "$sqlite3_has_pkg_config" = "no"; then > >+ AC_SEARCH_LIBS([sqlite3_open16], [sqlite3], > >+ [SQLITE3_LIBS="$LIBS";SQLITE3_CFLAGS="-I $srcdir/WebKitLibraries/WebCoreSQLite3"], > >+ [AC_MSG_ERROR([SQLite3 is required to enable Database support])]) > >+ fi > > Silly question, shouldn't you first try to use the headers that might come with > the library you just found with AC_SEARCH_LIBS? > Or are you assuming that if there's no sqlite pkg-config file the only library > you'll find is the one in WebKitLibraries? Yes. If there's no .pc file or there's no pkg-config binary, I'll just use what's in WebKitLibraries. (In reply to comment #4) > Did we ever get a report that someone wanted to use sqlite3 but didn't have > pkg-config files? Consider this a report. Landed as http://trac.webkit.org/changeset/49435. |