]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
kv: explicitly set DISABLE_JEMALLOC for rocksdb in Makefile.am.
authorKefu Chai <kchai@redhat.com>
Wed, 8 Jun 2016 03:32:10 +0000 (11:32 +0800)
committerKefu Chai <kchai@redhat.com>
Wed, 8 Jun 2016 04:06:31 +0000 (12:06 +0800)
we should disable rocksdb from using jemalloc even it is installed
in the building host. as it causes unresolved symbols if ceph is not
configured to use jemalloc and hence is not linked against it.

we can pass "DISABLE_JEMALLOC=0" to rocksdb's Makefile in hope to enable
jemalloc. because rocksdb checks for jemalloc using
```
ifndef DISABLE_JEMALLOC
```

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/kv/Makefile.am

index 5fcbe8012307c3c5dbaefab541956ae713f5d49d..789703472d9b4689e51c39d64ed0a1dceca3bac0 100644 (file)
@@ -26,14 +26,12 @@ NPROC = nproc
 endif
 
 # explicitly disable jemalloc unless we are using it.
-if WITH_JEMALLOC
-DISABLE_JEMALLOC = 0
-else
-DISABLE_JEMALLOC = 1
+if !WITH_JEMALLOC
+export DISABLE_JEMALLOC=1
 endif
 
 rocksdb/librocksdb.a:
-       cd rocksdb && CC="${CC}" CXX="${CXX}" EXTRA_CXXFLAGS="${ROCKSDBCXX_FLAGS}" PORTABLE=1 DISABLE_JEMALLOC="${DISABLE_JEMALLOC}" ${MAKE} -j$(shell ${NPROC}) static_lib
+       cd rocksdb && CC="${CC}" CXX="${CXX}" EXTRA_CXXFLAGS="${ROCKSDBCXX_FLAGS}" PORTABLE=1 ${MAKE} -j$(shell ${NPROC}) static_lib
 libkv_a_CXXFLAGS += -I rocksdb/include -fPIC
 libkv_a_SOURCES += kv/RocksDBStore.cc
 libkv_a_LIBADD += rocksdb/librocksdb.a