Improve gtest finding, support local checkout.

Also fix library dependency order for monolith test.

1 files changed, 16 insertions, 32 deletions

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 9450ab74..e6d9e2a2 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -14,8 +14,8 @@
 #
 ################################################################################
 
-cmake_minimum_required(VERSION 2.8.6)
-cmake_policy(VERSION 2.8.6)
+cmake_minimum_required(VERSION 3.1.0)
+cmake_policy(VERSION 3.1.0)
 project(toxcore)
 
 set(CMAKE_MODULE_PATH ${toxcore_SOURCE_DIR}/cmake)
@@ -66,11 +66,16 @@ enable_testing()
 
 set(CMAKE_MACOSX_RPATH ON)
 
-if(NOT MSVC)
-  # Set standard version for compiler.
-  add_cflag("-std=c99")
-  add_cxxflag("-std=c++11")
+# Set standard version for compiler.
+set(CMAKE_C_STANDARD 99)
+set(CMAKE_CXX_STANDARD 11)
+set(CMAKE_C_EXTENSIONS OFF)
+set(CMAKE_CXX_EXTENSIONS OFF)
+
+message(STATUS "Supported C compiler features = ${CMAKE_C_COMPILE_FEATURES}")
+message(STATUS "Supported C++ compiler features = ${CMAKE_CXX_COMPILE_FEATURES}")
 
+if(NOT MSVC)
   # Warn on non-ISO C.
   add_cflag("-pedantic")
 
@@ -436,35 +441,13 @@ install_module(toxcore DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/tox)
 #
 ################################################################################
 
-# Compile the GTest library.
-#
-if(EXISTS "/usr/src/gtest/src/gtest-all.cc")
-  add_library(gtest
-    /usr/src/gtest/src/gtest-all.cc
-    /usr/src/gtest/src/gtest_main.cc)
-  include_directories(/usr/src/gtest/include)
-  target_include_directories(gtest PRIVATE /usr/src/gtest)
-  check_cxx_compiler_flag("-w" HAVE_CXX_W QUIET)
-  check_cxx_compiler_flag("-Wno-global-constructors" HAVE_CXX_W_NO_GLOBAL_CONSTRUCTORS QUIET)
-  check_cxx_compiler_flag("-Wno-zero-as-null-pointer-constant" HAVE_CXX_W_NO_ZERO_AS_NULL_POINTER_CONSTANT QUIET)
-  if(HAVE_CXX_W)
-    set_target_properties(gtest PROPERTIES COMPILE_FLAGS "-w")
-  endif()
-  set(HAVE_GTEST TRUE)
-endif()
+include(CompileGTest)
 
 function(unit_test subdir target)
   if(HAVE_GTEST)
     add_executable(unit_${target}_test ${subdir}/${target}_test.cpp)
     target_link_modules(unit_${target}_test ${toxcore_SUBLIBS} gtest)
-    set(gtest_CFLAGS "")
-    if(HAVE_CXX_W_NO_GLOBAL_CONSTRUCTORS)
-      set(gtest_CFLAGS "${gtest_CFLAGS} -Wno-global-constructors")
-    endif()
-    if(HAVE_CXX_W_NO_ZERO_AS_NULL_POINTER_CONSTANT)
-      set(gtest_CFLAGS "${gtest_CFLAGS} -Wno-zero-as-null-pointer-constant")
-    endif()
-    set_target_properties(unit_${target}_test PROPERTIES COMPILE_FLAGS "${gtest_CFLAGS}")
+    set_target_properties(unit_${target}_test PROPERTIES COMPILE_FLAGS "${TEST_CXX_FLAGS}")
     add_test(NAME ${target} COMMAND ${CROSSCOMPILING_EMULATOR} unit_${target}_test)
   endif()
 endfunction()
@@ -518,10 +501,11 @@ if(BUILD_TOXAV)
     auto_tests/monolith_test.cpp
     ${ANDROID_CPU_FEATURES})
   target_link_modules(auto_monolith_test
-    ${toxcore_PKGCONFIG_LIBS}
     ${LIBSODIUM_LIBRARIES}
     ${OPUS_LIBRARIES}
-    ${VPX_LIBRARIES})
+    ${VPX_LIBRARIES}
+    ${toxcore_PKGCONFIG_LIBS}
+  )
   add_test(NAME monolith COMMAND ${CROSSCOMPILING_EMULATOR} auto_monolith_test)
 
   if(ANDROID_CPU_FEATURES)