PolytreeDE/polybar
1
0
Fork 0
mirror of https://github.com/polybar/polybar.git synced 2024-11-11 13:50:56 -05:00
Code Releases Activity

fix(gtest): prefer target_link_libraries

Browse source 
From commit 880896c6f4814f4c7798355a652dc6167be2b75f in googletest cmake
test configuration fails with errors like

CMake Error: install(EXPORT "gtestConfigInternal" ...) includes target
  "gtest" which requires target "i3ipc++" that is not in the export set.

The issue was that link_libraries also affected the googletest internal
targets that came after it making the whole thing fail.

This could also have been solved by moving link_libraries after the
configuration of googletest but according to the cmake documentation [1]
target_link_libraries should be prefered over link_libraries anyways.

[1]: https://cmake.org/cmake/help/latest/command/link_libraries.html

Fixes #1393
This commit is contained in:
patrick96 2018-08-31 12:24:00 +02:00 committed by NBonaparte
parent 2268136971
commit e794fe0d01
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
tests/CMakeLists.txt
View file

@ -2,7 +2,6 @@
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -g -fprofile-arcs -ftest-coverage -Wno-missing-field-initializers")
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -fprofile-arcs -ftest-coverage")
link_libraries(${libs})
include_directories(${dirs})
include_directories(${CMAKE_CURRENT_LIST_DIR})
@ -59,7 +58,7 @@ function(unit_test file tests)
add_executable(${name} unit_tests/${file}.cpp ${sources})
# Link against gmock (this automatically links against gtest)
target_link_libraries(${name} gmock_main)
target_link_libraries(${name} gmock_main ${libs})
add_test(NAME ${name} COMMAND ${name})