Updated test case CmakeLists.txt to work with newer version of CMAKE 02/137602/1
authorDavid Steele <david.steele@samsung.com>
Thu, 6 Jul 2017 15:56:01 +0000 (16:56 +0100)
committerDavid Steele <david.steele@samsung.com>
Thu, 6 Jul 2017 15:56:01 +0000 (16:56 +0100)
Change-Id: Iad48ae95621764d399ee6d1d5b727b9fbb15584c

automated-tests/src/dali-adaptor-internal/CMakeLists.txt
automated-tests/src/dali-adaptor/CMakeLists.txt
automated-tests/src/dali-platform-abstraction/CMakeLists.txt

index c41bc4d..ee37c4e 100644 (file)
@@ -38,7 +38,8 @@ PKG_CHECK_MODULES(${CAPI_LIB} REQUIRED
     dali-adaptor
     ecore
 )
     dali-adaptor
     ecore
 )
-SET(CMAKE_CXX_FLAGS  "${CMAKE_CXX_FLAGS} -O0 -ggdb --coverage -Wall -Werror" )
+
+ADD_COMPILE_OPTIONS( -O0 -ggdb --coverage -Wall -Werror )
 
 # Shouldn't have to do this!
 # But CMake's new auto-escape quote policy doesn't work right.
 
 # Shouldn't have to do this!
 # But CMake's new auto-escape quote policy doesn't work right.
@@ -71,7 +72,7 @@ INCLUDE_DIRECTORIES(
 ADD_EXECUTABLE(${EXEC_NAME} ${EXEC_NAME}.cpp ${TC_SOURCES})
 TARGET_LINK_LIBRARIES(${EXEC_NAME}
     ${${CAPI_LIB}_LIBRARIES}
 ADD_EXECUTABLE(${EXEC_NAME} ${EXEC_NAME}.cpp ${TC_SOURCES})
 TARGET_LINK_LIBRARIES(${EXEC_NAME}
     ${${CAPI_LIB}_LIBRARIES}
-    -lpthread
+    -lpthread --coverage
 )
 
 INSTALL(PROGRAMS ${EXEC_NAME}
 )
 
 INSTALL(PROGRAMS ${EXEC_NAME}
index 4ae66c4..ee64cb2 100644 (file)
@@ -43,8 +43,8 @@ PKG_CHECK_MODULES(${CAPI_LIB} REQUIRED
     ecore-x
 )
 
     ecore-x
 )
 
-SET(CMAKE_CXX_FLAGS  "${CMAKE_CXX_FLAGS} -O0 -ggdb --coverage -Wall -Werror")
-SET(CMAKE_CXX_FLAGS  "${CMAKE_CXX_FLAGS} ${${CAPI_LIB}_CFLAGS_OTHER}")
+ADD_COMPILE_OPTIONS( -O0 -ggdb --coverage -Wall -Werror )
+ADD_COMPILE_OPTIONS( ${${CAPI_LIB}_CFLAGS_OTHER} )
 
 ADD_DEFINITIONS(-DTEST_RESOURCE_DIR=\"${CMAKE_CURRENT_SOURCE_DIR}/../../resources\" )
 
 
 ADD_DEFINITIONS(-DTEST_RESOURCE_DIR=\"${CMAKE_CURRENT_SOURCE_DIR}/../../resources\" )
 
@@ -60,7 +60,8 @@ INCLUDE_DIRECTORIES(
 
 ADD_EXECUTABLE(${EXEC_NAME} ${EXEC_NAME}.cpp ${TC_SOURCES})
 TARGET_LINK_LIBRARIES(${EXEC_NAME}
 
 ADD_EXECUTABLE(${EXEC_NAME} ${EXEC_NAME}.cpp ${TC_SOURCES})
 TARGET_LINK_LIBRARIES(${EXEC_NAME}
-    ${${CAPI_LIB}_LIBRARIES}
+  ${${CAPI_LIB}_LIBRARIES}
+  --coverage
 )
 
 INSTALL(PROGRAMS ${EXEC_NAME}
 )
 
 INSTALL(PROGRAMS ${EXEC_NAME}
index 94b7bc3..5d04249 100644 (file)
@@ -31,7 +31,8 @@ PKG_CHECK_MODULES(${CAPI_LIB} REQUIRED
     dali-adaptor
 )
 
     dali-adaptor
 )
 
-SET(CMAKE_CXX_FLAGS  "${CMAKE_CXX_FLAGS} -O0 -ggdb --coverage -Wall -Werror" )
+ADD_COMPILE_OPTIONS(-O0 -ggdb --coverage -Wall -Werror)
+
 
 # Shouldn't have to do this!
 # But CMake's new auto-escape quote policy doesn't work right.
 
 # Shouldn't have to do this!
 # But CMake's new auto-escape quote policy doesn't work right.
@@ -53,7 +54,8 @@ INCLUDE_DIRECTORIES(
 
 ADD_EXECUTABLE(${EXEC_NAME} ${EXEC_NAME}.cpp ${TC_SOURCES})
 TARGET_LINK_LIBRARIES(${EXEC_NAME}
 
 ADD_EXECUTABLE(${EXEC_NAME} ${EXEC_NAME}.cpp ${TC_SOURCES})
 TARGET_LINK_LIBRARIES(${EXEC_NAME}
-    ${${CAPI_LIB}_LIBRARIES}
+  ${${CAPI_LIB}_LIBRARIES}
+  --coverage
 )
 
 INSTALL(PROGRAMS ${EXEC_NAME}
 )
 
 INSTALL(PROGRAMS ${EXEC_NAME}