Skip to content

Commit

Permalink
[VL] CMake configuration cleanup to remove variable VELOX_COMPONENTS_…
Browse files Browse the repository at this point in the history
…PATH (apache#8579)
  • Loading branch information
zhztheplayer authored Jan 21, 2025
1 parent df66e93 commit c8284e5
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions cpp/velox/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,6 @@ if(NOT DEFINED VELOX_BUILD_PATH)
endif()
endif()

set(VELOX_COMPONENTS_PATH "${VELOX_BUILD_PATH}/velox")

function(import_library TARGET_NAME LIB_PATH)
if(NOT EXISTS ${LIB_PATH})
message(FATAL_ERROR "Library does not exist: ${LIB_PATH}")
Expand Down Expand Up @@ -241,7 +239,7 @@ if(BUILD_TESTS)
${VELOX_BUILD_PATH}/velox/exec/tests/utils/libvelox_temp_path.a)
import_library(
facebook::velox::exec_test_lib
${VELOX_COMPONENTS_PATH}/exec/tests/utils/libvelox_exec_test_lib.a)
${VELOX_BUILD_PATH}/velox/exec/tests/utils/libvelox_exec_test_lib.a)
target_link_libraries(
facebook::velox::exec_test_lib
INTERFACE facebook::velox::vector_test_lib
Expand Down

0 comments on commit c8284e5

Please sign in to comment.