You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2022/12/03 03:12:09 UTC

[kudu] branch master updated: [thirdparty] skip building trace tools for RocksDB

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 6fdd8b453 [thirdparty] skip building trace tools for RocksDB
6fdd8b453 is described below

commit 6fdd8b4539b4aa46a059578e84b17d49da562846
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Fri Dec 2 17:58:17 2022 -0800

    [thirdparty] skip building trace tools for RocksDB
    
    This patch updates the list of components to build for RocksDB.  In
    particular, trace tools are no longer built.  It also fixes a misprint
    in specifying whether to build dynamic library.
    
    This is a follow-up to fab4c6fd30dc4bdfb52e854584e9433ef6c1f363.
    
    Change-Id: Idcc23ffe188d8bcc10e10cb3ca344f4f4bba8b9e
    Reviewed-on: http://gerrit.cloudera.org:8080/19316
    Reviewed-by: Yingchun Lai <ac...@gmail.com>
    Tested-by: Kudu Jenkins
---
 thirdparty/build-definitions.sh | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/thirdparty/build-definitions.sh b/thirdparty/build-definitions.sh
index 0610c558c..e3d44416c 100644
--- a/thirdparty/build-definitions.sh
+++ b/thirdparty/build-definitions.sh
@@ -1163,11 +1163,13 @@ build_rocksdb() {
   CFLAGS="$EXTRA_CFLAGS -fPIC" \
     CXXFLAGS="$EXTRA_CXXFLAGS -fPIC" \
     cmake \
-    -ROCKSDB_BUILD_SHARED=ON \
+    -DROCKSDB_BUILD_SHARED=ON \
     -DFAIL_ON_WARNINGS=OFF \
     -DWITH_BENCHMARK_TOOLS=OFF \
-    -DWITH_TOOLS=OFF \
     -DWITH_CORE_TOOLS=OFF \
+    -DWITH_TOOLS=OFF \
+    -DWITH_TRACE_TOOLS=OFF \
+    -DWITH_JNI=OFF \
     -DWITH_LZ4=ON \
     -DWITH_ZSTD=OFF \
     -DWITH_SNAPPY=ON \