Skip to content

Commit

Permalink
build: Removed GTest::gtest_main from CMakeLists.txt for velox_simple…
Browse files Browse the repository at this point in the history
…_aggregate_test (#11668)

Summary:
Removed GTest::gtest_main from CMakeLists.txt for velox_simple_aggregate_test.

```
add_executable(velox_simple_aggregate_test SimpleAggregateAdapterTest.cpp
                                           Main.cpp)
```

Since Main.cpp is already included here, GTest::gtest_main is not needed in target_link_libraries.

Pull Request resolved: #11668

Reviewed By: kagamiori

Differential Revision: D66690701

Pulled By: xiaoxmeng

fbshipit-source-id: 51c5536de45bfd15abdf7a83f46c95f09846a976
  • Loading branch information
minhancao authored and facebook-github-bot committed Dec 3, 2024
1 parent fc5aa37 commit 4dd6499
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions velox/exec/tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -284,8 +284,9 @@ target_link_libraries(
velox_simple_aggregate
velox_exec
velox_functions_aggregates_test_lib
GTest::gtest
GTest::gtest_main)
GTest::gtest)

add_test(velox_simple_aggregate_test velox_simple_aggregate_test)

if(VELOX_ENABLE_BENCHMARKS)
add_library(velox_spiller_join_benchmark_base JoinSpillInputBenchmarkBase.cpp
Expand Down

0 comments on commit 4dd6499

Please sign in to comment.