[CMAKE]
authorAmine Khaldi <amine.khaldi@reactos.org>
Mon, 14 Feb 2011 10:32:46 +0000 (10:32 +0000)
committerAmine Khaldi <amine.khaldi@reactos.org>
Mon, 14 Feb 2011 10:32:46 +0000 (10:32 +0000)
- Get rid of the redundancy in setting msvc flags, and set /ZI instead of /Zi for both host and target modules.
- Get rid of the redundancy in setting the default behaviour of the FIND_XXX() commands between toolchain files.

svn path=/branches/cmake-bringup/; revision=50688

CMakeLists.txt
cmake/Platform/Windows-cl.cmake
msc.cmake
toolchain-mingw32.cmake
toolchain-msvc.cmake

index d04a92c..01a108a 100644 (file)
@@ -61,6 +61,13 @@ if(NOT CMAKE_CROSSCOMPILING)
 
 else()
 
+    # adjust the default behaviour of the FIND_XXX() commands:
+    # search headers and libraries in the target environment, search
+    # programs in the host environment
+    set(CMAKE_FIND_ROOT_PATH_MODE_PROGRAM NEVER)
+    set(CMAKE_FIND_ROOT_PATH_MODE_LIBRARY ONLY)
+    set(CMAKE_FIND_ROOT_PATH_MODE_INCLUDE ONLY)
+
     # Do some cleanup
     file(REMOVE
         ${REACTOS_BINARY_DIR}/dependencies.graphml
index 0761306..a14ee48 100644 (file)
@@ -194,7 +194,7 @@ IF("${MSVC_VERSION}" GREATER 1599)
 ENDIF()\r
 \r
 # default to Debug builds\r
-IF(MSVC_VERSION GREATER 1310)\r
+if(MSVC_VERSION GREATER 1310)\r
   # for 2005 make sure the manifest is put in the dll with mt\r
 #  SET(CMAKE_CXX_CREATE_SHARED_LIBRARY "<CMAKE_COMMAND> -E vs_link_dll ${CMAKE_CXX_CREATE_SHARED_LIBRARY}")\r
 #  SET(CMAKE_CXX_CREATE_SHARED_MODULE "<CMAKE_COMMAND> -E vs_link_dll ${CMAKE_CXX_CREATE_SHARED_MODULE}")\r
@@ -205,15 +205,15 @@ IF(MSVC_VERSION GREATER 1310)
 #  SET(CMAKE_CXX_LINK_EXECUTABLE "<CMAKE_COMMAND> -E vs_link_exe ${CMAKE_CXX_LINK_EXECUTABLE}")\r
 #  SET(CMAKE_C_LINK_EXECUTABLE "<CMAKE_COMMAND> -E vs_link_exe ${CMAKE_C_LINK_EXECUTABLE}")\r
 \r
-  SET(CMAKE_BUILD_TYPE_INIT Debug)\r
+    set(CMAKE_BUILD_TYPE_INIT Debug)\r
 \r
-  IF(ENV{DDKBUILDENV} OR NOT CMAKE_CROSSCOMPILING)\r
-    SET (CMAKE_CXX_FLAGS_DEBUG_INIT "/D_DEBUG /MTd /Zi /Ob0 /Od")\r
-    SET (CMAKE_C_FLAGS_DEBUG_INIT "/D_DEBUG /Zi  /Ob0 /Od")\r
-  ELSE()\r
-    SET (CMAKE_CXX_FLAGS_DEBUG_INIT "/D_DEBUG /Zi /Ob0 /Od")\r
-    SET (CMAKE_C_FLAGS_DEBUG_INIT "/D_DEBUG /Zi  /Ob0 /Od")\r
-  ENDIF()\r
+    if(ENV{DDKBUILDENV} OR NOT CMAKE_CROSSCOMPILING)\r
+        set(CMAKE_CXX_FLAGS_DEBUG_INIT "/D_DEBUG /MTd /ZI /Ob0 /Od")\r
+        set(CMAKE_C_FLAGS_DEBUG_INIT "/D_DEBUG /ZI /Ob0 /Od")\r
+    else()\r
+        set(CMAKE_CXX_FLAGS_DEBUG_INIT "/D_DEBUG /ZI /Ob0 /Od")\r
+        set(CMAKE_C_FLAGS_DEBUG_INIT "/D_DEBUG /ZI /Ob0 /Od")\r
+    endif()\r
 \r
   SET (CMAKE_CXX_FLAGS_INIT "/DWIN32 /D_WINDOWS /W3 /Zm1000 /EHsc")\r
   SET (CMAKE_CXX_FLAGS_MINSIZEREL_INIT "/O1 /Ob1 /D NDEBUG")\r
index 95dae34..b0c3750 100644 (file)
--- a/msc.cmake
+++ b/msc.cmake
@@ -13,7 +13,7 @@ elseif(OPTIMIZE STREQUAL "5")
 endif()\r
 endif()\r
 \r
-add_definitions(/X /GR- /GS- /Zl /Zi)\r
+add_definitions(/X /GR- /GS- /Zl)\r
 add_definitions(-Dinline=__inline -D__STDC__=1)\r
 \r
 if(${_MACHINE_ARCH_FLAG} MATCHES X86)\r
@@ -130,9 +130,6 @@ macro(set_unicode)
    set(IS_UNICODE 1)\r
 endmacro()\r
 \r
-set(CMAKE_C_FLAGS_DEBUG_INIT "/D_DEBUG /MDd /Zi  /Ob0 /Od")\r
-set(CMAKE_CXX_FLAGS_DEBUG_INIT "/D_DEBUG /MDd /Zi /Ob0 /Od")\r
-\r
 macro(set_rc_compiler)\r
 # dummy, this workaround is only needed in mingw due to lack of RC support in cmake\r
 endmacro()\r
index b263d2d..a6bae6c 100644 (file)
@@ -48,10 +48,3 @@ if(ARCH MATCHES i386)
 elseif(ARCH MATCHES amd64)
     set(CMAKE_SHARED_LINKER_FLAGS_INIT "-nodefaultlibs -nostdlib -Wl,--enable-auto-image-base -Wl,--disable-auto-import")
 endif()
-
-# adjust the default behaviour of the FIND_XXX() commands:
-# search headers and libraries in the target environment, search
-# programs in the host environment
-set(CMAKE_FIND_ROOT_PATH_MODE_PROGRAM NEVER)
-set(CMAKE_FIND_ROOT_PATH_MODE_LIBRARY ONLY)
-set(CMAKE_FIND_ROOT_PATH_MODE_INCLUDE ONLY)
index 6c781b4..6687fba 100644 (file)
@@ -20,12 +20,6 @@ set(CMAKE_ASM_COMPILE_OBJECT
     "<CMAKE_ASM_COMPILER> /nologo /Cp /Fo<OBJECT> /c /Ta <OBJECT>.tmp")\r
 \r
 set(CMAKE_C_FLAGS_INIT "/DWIN32 /D_WINDOWS /W1 /Zm1000")\r
-set(CMAKE_C_FLAGS_DEBUG_INIT "/D_DEBUG /MDd /Zi  /Ob0 /Od")\r
-set(CMAKE_CXX_FLAGS_DEBUG_INIT "/D_DEBUG /MDd /Zi /Ob0 /Od")\r
-\r
-set(CMAKE_FIND_ROOT_PATH_MODE_PROGRAM NEVER)\r
-set(CMAKE_FIND_ROOT_PATH_MODE_LIBRARY ONLY)\r
-set(CMAKE_FIND_ROOT_PATH_MODE_INCLUDE ONLY)\r
 \r
 set(CMAKE_C_STANDARD_LIBRARIES "" CACHE INTERNAL "")\r
 \r