[OE-core] Pseudo patch/change, sanity check requested

[Peter Seebach]

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 <peter.seebach at windriver.com>
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 <peter.seebach at windriver.com>

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.