fix std::variant detection

having variant header is not enough, c++17 has to be enabled to have support for std::variant
shogun-toolbox · May 22, 2018 · 21e932b · 21e932b
1 parent 72c23c1
commit 21e932b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -47,9 +47,6 @@ if (MSVC)
 	endif()
 endif()
 
-include(CheckIncludeFileCXX)
-CHECK_INCLUDE_FILE_CXX("variant" HAVE_STD_VARIANT)
-
 ### FIXME: remove these flags when the codebase
 # is cleared up
 set(HAVE_CXX11 ON)

diff --git a/src/shogun/CMakeLists.txt b/src/shogun/CMakeLists.txt
@@ -325,6 +325,7 @@ ENDIF()
 #### LAPACK
 include(ShogunFindLAPACK)
 
+CHECK_CXX_SOURCE_COMPILES("#include <variant>\n int main(int argc, char** argv) { return 0; }" HAVE_STD_VARIANT)
 # variant
 IF (NOT HAVE_STD_VARIANT)
   include(external/variant)