From 59a5f333fa3c3e9ef3a9c9d3fded43d23cfce911 Mon Sep 17 00:00:00 2001 From: Sheng Mao Date: Tue, 9 Feb 2021 20:14:58 -0700 Subject: [PATCH] cmake: don't pass ccache argument to RocksDB build RocksDB's cmake has a similar logic to turn on ccache as the main cmake. Currently BuildRocksDB.cmake explicitly passes ccache to RocksDB cmake arguments as CMAKE_CXX_COMPILER_LAUNCHER. This causes a compilation error on ccache 4.0, especially in make -j mode. ccache: error: Recursive invocation (the name of the ccache binary must be "ccache") This commit lets cmake from RocksDB decide RULE_LAUNCH_COMPILE by itself. Signed-off-by: Sheng Mao --- cmake/modules/BuildRocksDB.cmake | 3 --- 1 file changed, 3 deletions(-) diff --git a/cmake/modules/BuildRocksDB.cmake b/cmake/modules/BuildRocksDB.cmake index 58ce26a6aca..5ab2e8e49d8 100644 --- a/cmake/modules/BuildRocksDB.cmake +++ b/cmake/modules/BuildRocksDB.cmake @@ -16,9 +16,6 @@ function(build_rocksdb) list(APPEND rocksdb_INTERFACE_LINK_LIBRARIES JeMalloc::JeMalloc) endif() - if (WITH_CCACHE AND CCACHE_FOUND) - list(APPEND rocksdb_CMAKE_ARGS -DCMAKE_CXX_COMPILER_LAUNCHER=ccache) - endif() list(APPEND rocksdb_CMAKE_ARGS -DCMAKE_CXX_COMPILER=${CMAKE_CXX_COMPILER}) list(APPEND rocksdb_CMAKE_ARGS -DWITH_SNAPPY=${SNAPPY_FOUND}) -- 2.39.5