Merge pull request #2 from frederikvannoote/feature/noAllWarningsForV…

…isualStudio It is very hard to solve all warnings Visual Studio is generating. Le…
frederikvannoote · Feb 10, 2017 · 2dd971f · 2dd971f
2 parents ba3d620 + f26d51d
commit 2dd971f
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/CommonConfig.cmake b/CommonConfig.cmake
@@ -47,7 +47,10 @@ list(APPEND CMAKE_PREFIX_PATH ${CMAKE_BINARY_DIR}/local-exports)
 
 # Compiler flags
 # --------------
-add_definitions(-Wall -fvisibility=hidden)
+if (NOT "${CMAKE_CXX_COMPILER_ID}" STREQUAL "MSVC")
+    # not using Visual Studio C++
+    add_definitions(-Wall -fvisibility=hidden)
+endif()
 set(CMAKE_CXX_STANDARD 11)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)