Skip to contentSkip to conversation

package/thrift: force boost libdir to staging/usr/lib/

PATCH New Romain Naour 2 participants 2 messages Feb 23, 2015 – Mar 8, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/thrift: force boost libdir to staging/usr/lib/

Libtool append the sysroot path (lt_sysroot) with the libdir value found in the libstdc++.la file only with bfin toolchain.

Path/to/sysroot/opt/uClinux/bfin-linux-uclibc/lib/gcc/bfin-linux-uclibc/4.3.5/libstdc++.so

The issue disappears when --with-boost=$(STAGING_DIR) is removed. So instead of using --with-boost=$(STAGING_DIR), use --with-boost-libdir=$(STAGING_DIR)/usr/lib to give the path to boost's libraries.

Fixes:
http://autobuild.buildroot.net/results/ae2/ae274693b953dd569c3ff63c532e2fc9553b5002/

And many more

---
 package/thrift/thrift.mk | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
package/thrift/thrift.mk +2 -1
index 559bd28..094f8d0 100644
@@ -11,7 +11,8 @@ THRIFT_INSTALL_STAGING = YES
 HOST_THRIFT_DEPENDENCIES = host-boost host-libevent host-openssl host-pkgconf \
 	host-zlib host-bison host-flex
 THRIFT_CONF_OPTS = --with-sysroot=$(STAGING_DIR) \
-	--with-boost=$(STAGING_DIR) \
+	--with-boost \
+	--with-boost-libdir=$(STAGING_DIR)/usr/lib \
 	--disable-tests \
 	--disable-tutorial
 HOST_THRIFT_CONF_OPTS = --with-sysroot=$(HOST_DIR) \
Signature · 1 line
1.9.3
#1 lore raw

PATCH 1/1 package/thrift: force boost libdir to staging/usr/lib/

Dear Romain Naour,

Quoted 10 lines · Romain Naour

Libtool append the sysroot path (lt_sysroot) with the libdir value found in the libstdc++.la file only with bfin toolchain.

Path/to/sysroot/opt/uClinux/bfin-linux-uclibc/lib/gcc/bfin-linux-uclibc/4.3.5/libstdc++.so

The issue disappears when --with-boost=$(STAGING_DIR) is removed. So instead of using --with-boost=$(STAGING_DIR), use --with-boost-libdir=$(STAGING_DIR)/usr/lib to give the path to boost's libraries.

Quoted 6 lines

Fixes:
http://autobuild.buildroot.net/results/ae2/ae274693b953dd569c3ff63c532e2fc9553b5002/

And many more

The proposed solution is a little bit of a hack, but libtool is just a pain, and the solution you propose is not invasive at all, so I applied this.

Thanks a lot for spending on this long standing issue!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com