-
Notifications
You must be signed in to change notification settings - Fork 19
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.
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
split cmakelist #384
base: master
Are you sure you want to change the base?
split cmakelist #384
Conversation
a6cf554
to
6253895
Compare
|
||
target_link_libraries(LoadTest testlib gtest gmock) | ||
add_subdirectory(test) | ||
set_target_properties(UnitTest PROPERTIES LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR} ARCHIVE_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR} RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is to make Test executables end up in build top folder.
However, maybe we should consider allowing the build folder to be structured as cmake default:
build
bin
_deps
libs
and put the binaries in bin folder
include(FetchContent) | ||
|
||
# Download and unpack googletest at configure time | ||
configure_file(CMakeLists-gtest.txt.in googletest-download/CMakeLists.txt) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be done with fetchContent but it would require the cmake to be upgraded on all docker images first.
6d6272d
to
be2c5aa
Compare
be2c5aa
to
57c4f98
Compare
Moves the test build into its own cmakelist.
Upgraded gtest to 1.15.2
This PR should be updated once the new docker images are in use with make 3.30