Okay, this is a little odd. We had a build failure of pseudo for a 64-bit target. I looked. I found an easy fix.
I'm sending this out to the list before I check it in, for one simple reason: It is absolutely obvious to me that, without this fix, the --enable-static-sqlite feature should never have worked on 64-bit systems. Obviously, it's been in use for months. Which means there's SOMETHING going on which I don't understand. And I am soliciting feedback because I am not comfortable checking in a fix I don't understand, and I recognize that changes to pseudo are relatively high risk. What I'm looking for: 1. Reports from anyone who's been bitten by this. 2. Feedback on how it could be that the build system has worked despite the obvious bug. The bug is simply that --enable-static-sqlite has been hard-coding the path to $(SQLITE)/lib/libsqlite3.a. This obviously can't work on systems where the library directory is lib64. So why haven't we seen it before last week? -s --- commit b6c2409d786e53f62accc1c7a6538f39dd0ab601 Author: Peter Seebach <[email protected]> Date: Fri Nov 2 19:55:57 2012 -0500 Makefile.in/configure: Use $(LIB), not hardcoded lib, for sqlite It turns out that the -L usage mostly doesn't matter (usually something else has requested the right directory, or it's the default), but the explicit path to libsqlite3.a hardcoded "lib", and on some systems it should be something else, such as "lib64". Solution: Use $(LIB) for that directory. Note that this may not resolve things if, say, you're doing MIPS n32 on a target where that lives in /usr/lib32, but I think in that case you'd be specifying $libdir, so it should still work out. Also added --with-static-sqlite=/path as an option in case people need to further outsmart this. Signed-off-by: Peter Seebach <[email protected]> diff --git a/Makefile.in b/Makefile.in index 83fc89f..56ea5e2 100644 --- a/Makefile.in +++ b/Makefile.in @@ -38,7 +38,7 @@ CFLAGS_BASE=-pipe -std=gnu99 -Wall -W -Wextra CFLAGS_CODE=-fPIC -D_LARGEFILE64_SOURCE -D_ATFILE_SOURCE $(ARCH_FLAGS) CFLAGS_DEFS=-DPSEUDO_PREFIX='"$(PREFIX)"' -DPSEUDO_SUFFIX='"$(SUFFIX)"' -DPSEUDO_BINDIR='"$(BIN)"' -DPSEUDO_LIBDIR='"$(LIB)"' -DPSEUDO_LOCALSTATEDIR='"$(LOCALSTATE)"' -DPSEUDO_VERSION='"$(VERSION)"' CFLAGS_DEBUG=-O2 -g -CFLAGS_SQL=-L$(SQLITE)/lib -I$(SQLITE)/include $(RPATH) +CFLAGS_SQL=-L$(SQLITE)/$(LIB) -I$(SQLITE)/include $(RPATH) CFLAGS_PSEUDO=$(CFLAGS_BASE) $(CFLAGS_CODE) $(CFLAGS_DEFS) \ $(CFLAGS_DEBUG) $(CFLAGS_SQL) diff --git a/configure b/configure index 4a41943..c536f58 100755 --- a/configure +++ b/configure @@ -37,6 +37,7 @@ usage() echo >&2 " [--suffix=...]" echo >&2 " [--with-sqlite=...]" echo >&2 " [--enable-static-sqlite]" + echo >&2 " [--with-static-sqlite=...]" echo >&2 " [--with-rpath=...|--without-rpath]" echo >&2 " [--cflags='']" echo >&2 " [--bits=32|64]" @@ -58,14 +59,19 @@ do --libdir=*) opt_libdir=${arg#--libdir=} ;; + --with-static-sqlite=*) + opt_sqlite_ldarg=${arg#--with-static-sqlite=} + sqlite_ldarg=$opt_sqlite_ldarg + use_maybe_rpath=false + ;; --enable-static-sqlite) - sqlite_ldarg='$(SQLITE)/lib/libsqlite3.a' + sqlite_ldarg='$(SQLITE)/$(LIB)/libsqlite3.a' use_maybe_rpath=false ;; --with-sqlite=*) opt_sqlite=${arg#--with-sqlite=} # assign new value if unset - maybe_rpath='-Wl,-R$(SQLITE)/lib' + maybe_rpath='-Wl,-R$(SQLITE)/$(LIB)' ;; --without-rpath) opt_rpath='' -- Listen, get this. Nobody with a good compiler needs to be justified. _______________________________________________ Openembedded-core mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
