Skip to content

Fix linking errors in the test suite when disabling C adaptor #420

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rapids-bot merged 4 commits into from
Sep 27, 2025
Merged

Fix linking errors in the test suite when disabling C adaptor #420

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a04bd59
fix linking errors when disabling c adaptor
nguidotti Sep 26, 2025
afe68ec
fixed styling
nguidotti Sep 26, 2025
e30b259
Merge branch 'branch-25.10' into fix-linking-errors
nguidotti Sep 26, 2025
81e5f80
Merge branch 'branch-25.10' into fix-linking-errors
nguidotti Sep 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
78 changes: 40 additions & 38 deletions cpp/tests/linear_programming/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,46 +26,48 @@ ConfigureTest(PDLP_TEST
# - C API Tests----------------------------------------------------------------------
# The C API tests require a separate library to be linked against. So we don't use the ConfigureTest macro.

add_library(c_api_tester "${CMAKE_CURRENT_SOURCE_DIR}/c_api_tests/c_api_test.c")
target_include_directories(c_api_tester
PRIVATE
${CUOPT_SOURCE_DIR}/include
PUBLIC
"$<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/include>"
)
target_include_directories(c_api_tester
PRIVATE
"${CMAKE_CURRENT_SOURCE_DIR}/c_api_tests"
)
if (NOT SKIP_C_PYTHON_ADAPTERS)
add_library(c_api_tester "${CMAKE_CURRENT_SOURCE_DIR}/c_api_tests/c_api_test.c")
target_include_directories(c_api_tester
PRIVATE
${CUOPT_SOURCE_DIR}/include
PUBLIC
"$<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/include>"
)
target_include_directories(c_api_tester
PRIVATE
"${CMAKE_CURRENT_SOURCE_DIR}/c_api_tests"
)

add_executable(C_API_TEST "${CMAKE_CURRENT_SOURCE_DIR}/c_api_tests/c_api_tests.cpp")
add_executable(C_API_TEST "${CMAKE_CURRENT_SOURCE_DIR}/c_api_tests/c_api_tests.cpp")

target_include_directories(C_API_TEST
PRIVATE
"${CUOPT_TEST_DIR}/../src"
"${CUOPT_TEST_DIR}/../libmps_parser/src"
"${CUOPT_TEST_DIR}"
"${CMAKE_CURRENT_SOURCE_DIR}/c_api_tests"
)
target_include_directories(C_API_TEST
PRIVATE
"${CUOPT_TEST_DIR}/../src"
"${CUOPT_TEST_DIR}/../libmps_parser/src"
"${CUOPT_TEST_DIR}"
"${CMAKE_CURRENT_SOURCE_DIR}/c_api_tests"
)

target_link_libraries(C_API_TEST
PRIVATE
mps_parser
cuopt
cuopttestutils
c_api_tester
GTest::gmock
GTest::gmock_main
GTest::gtest
GTest::gtest_main
${CUOPT_PRIVATE_CUDA_LIBS}
)
target_link_libraries(C_API_TEST
PRIVATE
mps_parser
cuopt
cuopttestutils
c_api_tester
GTest::gmock
GTest::gmock_main
GTest::gtest
GTest::gtest_main
${CUOPT_PRIVATE_CUDA_LIBS}
)

add_test(NAME C_API_TEST COMMAND C_API_TEST)
add_test(NAME C_API_TEST COMMAND C_API_TEST)

install(
TARGETS C_API_TEST
COMPONENT testing
DESTINATION bin/gtests/libcuopt
EXCLUDE_FROM_ALL
)
install(
TARGETS C_API_TEST
COMPONENT testing
DESTINATION bin/gtests/libcuopt
EXCLUDE_FROM_ALL
)
endif()