Skip to content

Commit 8877219

Browse files
bdicemeta-codesync[bot]
authored andcommitted
fix(cudf): Add missing Folly::folly dependency to velox_cudf_config_test (#16319)
Summary: `Main.cpp` includes `folly/Unit.h` and `folly/init/Init.h` but the `velox_cudf_config_test` target was not linking `Folly::folly`, causing a missing header error when folly is built as `BUNDLED`. Pull Request resolved: #16319 Reviewed By: jainxrohit Differential Revision: D92843853 Pulled By: xiaoxmeng fbshipit-source-id: e3e4c9f1f7f911266975c48bf2c9766ecb461e13
1 parent 80ad4c1 commit 8877219

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

velox/experimental/cudf/tests/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ target_link_libraries(
195195
gtest_main
196196
)
197197

198-
target_link_libraries(velox_cudf_config_test velox_cudf_exec gtest gtest_main)
198+
target_link_libraries(velox_cudf_config_test velox_cudf_exec Folly::folly gtest gtest_main)
199199

200200
target_link_libraries(
201201
velox_cudf_expression_selection_test

0 commit comments

Comments
 (0)