Define CXXFLAGS only on UNIX
authorBrendan Shanks <bshanks@audiovox.com>
Fri, 21 Mar 2014 18:07:40 +0000 (11:07 -0700)
committerJustin Dickow <jjdickow@gmail.com>
Wed, 26 Mar 2014 20:17:00 +0000 (16:17 -0400)
Define CXXFLAGS only on UNIX(/OS X/Linux/QNX), MSVC doesn't recognize any of these gcc options
Also move the Linux/QNX/Mac specific sections inside if(UNIX) to make it more clear

SDL_Core/CMakeLists.txt

index db6e54e..3466830 100644 (file)
@@ -41,8 +41,6 @@ set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_SOURCE_DIR}/cmake/Modules/")
 # flag is handled by CMake
 set(CMAKE_INSTALL_PREFIX .)
 set(ARCHIVE_OUTPUT_DIRECTORY ./bin)
-set(CMAKE_CXX_FLAGS "-g3 -ggdb3 -std=gnu++0x -Werror=return-type -Wuninitialized")
-set(CMAKE_CXX_FLAGS_RELEASE "-s -O2") #It will be appended to CMAKE_CXX_FLAGS in release
 
 option(EXTENDED_MEDIA_MODE "Turn on and off extended Madia Manager features relates to PulseAudio A2DP and GStreamer" OFF)
 option(BUILD_SHARED_LIBS "Build all libraries as shared (if ON) or static (if OFF)" OFF)
@@ -54,21 +52,25 @@ option(BUILD_BACKTRACE_SUPPORT "backtrace support" ON)
 
 
 if (UNIX)
-    add_definitions(-DOS_POSIX)
-endif(UNIX)
-if (CMAKE_SYSTEM_NAME STREQUAL "Linux")
-  add_definitions(-DOS_LINUX)
-elseif(CMAKE_SYSTEM_NAME STREQUAL "QNX")
-  add_definitions(-DOS_QNX)
-  SET(BUILD_BT_SUPPORT OFF)
-  SET(BUILD_AVAHI_SUPPORT OFF)
-  SET(BUILD_BACKTRACE_SUPPORT OFF)
-  SET(EXTENDED_MEDIA_MODE OFF)
-elseif(CMAKE_SYSTEM_NAME STREQUAL Darwin)
+  add_definitions(-DOS_POSIX)
+
+  set(CMAKE_CXX_FLAGS "-g3 -ggdb3 -std=gnu++0x -Werror=return-type -Wuninitialized")
+  set(CMAKE_CXX_FLAGS_RELEASE "-s -O2") #It will be appended to CMAKE_CXX_FLAGS in release
+
+  if (CMAKE_SYSTEM_NAME STREQUAL "Linux")
+    add_definitions(-DOS_LINUX)
+  elseif(CMAKE_SYSTEM_NAME STREQUAL "QNX")
+    add_definitions(-DOS_QNX)
+    SET(BUILD_BT_SUPPORT OFF)
+    SET(BUILD_AVAHI_SUPPORT OFF)
+    SET(BUILD_BACKTRACE_SUPPORT OFF)
+    SET(EXTENDED_MEDIA_MODE OFF)
+  elseif(CMAKE_SYSTEM_NAME STREQUAL Darwin)
     add_definitions(-DOS_MACOSX)
     #compile 32-bit 64-bit is having an issue
     SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -arch i386")
-endif()
+  endif()
+endif(UNIX)
 
 if (BUILD_USB_SUPPORT)
   add_definitions(-DUSB_SUPPORT)