[CMAKE]
authorTimo Kreuzer <timo.kreuzer@reactos.org>
Mon, 8 Nov 2010 12:24:33 +0000 (12:24 +0000)
committerTimo Kreuzer <timo.kreuzer@reactos.org>
Mon, 8 Nov 2010 12:24:33 +0000 (12:24 +0000)
- add new macro add_linkerflag to avoid code duplication
- add new macro set_pdef_file, replacing the use of pdef2def, which is neccessary to resolve issues with MSVC. cmake doesn't handle def files as source files very well, when they are not in the current source directory.

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

gcc.cmake
msc.cmake

index 9b64008..ed83304 100644 (file)
--- a/gcc.cmake
+++ b/gcc.cmake
@@ -32,12 +32,8 @@ add_definitions(-Wall -Wno-char-subscripts -Wpointer-arith -Wno-multichar -Wno-e
 add_definitions(-Os -fno-strict-aliasing -ftracer -momit-leaf-frame-pointer -mpreferred-stack-boundary=2 -fno-set-stack-executable -fno-optimize-sibling-calls)
 
 # Macros
-macro(set_entrypoint MODULE ENTRYPOINT)
-    if(${ENTRYPOINT} STREQUAL "0")
-        set(NEW_LINKER_FLAGS "-Wl,-entry,0")
-    else()
-        set(NEW_LINKER_FLAGS "-Wl,-entry,_${ENTRYPOINT}")
-    endif()
+macro(add_linkerflag MODULE _flag)
+    set(NEW_LINKER_FLAGS ${_flag})
     get_target_property(LINKER_FLAGS ${MODULE} LINK_FLAGS)
     if(LINKER_FLAGS)
         set(NEW_LINKER_FLAGS "${LINKER_FLAGS} ${NEW_LINKER_FLAGS}")
@@ -45,13 +41,16 @@ macro(set_entrypoint MODULE ENTRYPOINT)
     set_target_properties(${MODULE} PROPERTIES LINK_FLAGS ${NEW_LINKER_FLAGS})
 endmacro()
 
-macro(set_subsystem MODULE SUBSYSTEM)
-    set(NEW_LINKER_FLAGS "-Wl,--subsystem,${SUBSYSTEM}")
-    get_target_property(LINKER_FLAGS ${MODULE} LINK_FLAGS)
-    if(LINKER_FLAGS)
-        set(NEW_LINKER_FLAGS "${LINKER_FLAGS} ${NEW_LINKER_FLAGS}")
+macro(set_entrypoint MODULE ENTRYPOINT)
+    if(${ENTRYPOINT} STREQUAL "0")
+        add_linkerflag(${MODULE} "-Wl,-entry,0")
+    else()
+        add_linkerflag(${MODULE} "-Wl,-entry,_${ENTRYPOINT}")
     endif()
-    set_target_properties(${MODULE} PROPERTIES LINK_FLAGS ${NEW_LINKER_FLAGS})
+endmacro()
+
+macro(set_subsystem MODULE SUBSYSTEM)
+    add_linkerflag(${MODULE} "-Wl,--subsystem,${SUBSYSTEM}")
 endmacro()
 
 macro(set_image_base MODULE IMAGE_BASE)
@@ -167,8 +166,17 @@ macro(pdef2def _pdef_file)
         DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/${_pdef_file})
     set_source_files_properties(${CMAKE_CURRENT_BINARY_DIR}/${_file}.def
         PROPERTIES GENERATED TRUE EXTERNAL_OBJECT TRUE)
+    add_custom_target(
+        ${_file}_def
+        DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/${_file}.def)
 endmacro(pdef2def _pdef_file)
 
+macro(set_pdef_file _module _pdef_file)
+    pdef2def(${_pdef_file})
+    get_filename_component(_file ${_pdef_file} NAME_WE)
+    target_link_libraries(${_module} "${CMAKE_CURRENT_BINARY_DIR}/${_file}.def")
+endmacro()
+
 #pseh lib, needed with mingw
 set(PSEH_LIB "pseh")
 
index 553c970..506a7f5 100644 (file)
--- a/msc.cmake
+++ b/msc.cmake
@@ -23,13 +23,8 @@ link_directories("${REACTOS_BINARY_DIR}/importlibs" ${REACTOS_BINARY_DIR}/lib/3r
 \r
 set(CMAKE_RC_CREATE_SHARED_LIBRARY "<CMAKE_C_COMPILER> <CMAKE_SHARED_LIBRARY_C_FLAGS> <LINK_FLAGS> <CMAKE_SHARED_LIBRARY_CREATE_C_FLAGS> -o <TARGET> <OBJECTS> <LINK_LIBRARIES>")\r
 \r
-\r
-macro(set_entrypoint MODULE ENTRYPOINT)\r
-    if(${ENTRYPOINT} STREQUAL "0")\r
-        set(NEW_LINKER_FLAGS "/ENTRY:0")\r
-    else()\r
-        set(NEW_LINKER_FLAGS "/ENTRY:${ENTRYPOINT}")\r
-    endif()\r
+macro(add_linkerflag MODULE _flag)\r
+    set(NEW_LINKER_FLAGS ${_flag})\r
     get_target_property(LINKER_FLAGS ${MODULE} LINK_FLAGS)\r
     if(LINKER_FLAGS)\r
         set(NEW_LINKER_FLAGS "${LINKER_FLAGS} ${NEW_LINKER_FLAGS}")\r
@@ -37,22 +32,20 @@ macro(set_entrypoint MODULE ENTRYPOINT)
     set_target_properties(${MODULE} PROPERTIES LINK_FLAGS ${NEW_LINKER_FLAGS})\r
 endmacro()\r
 \r
-macro(set_subsystem MODULE SUBSYSTEM)\r
-    set(NEW_LINKER_FLAGS "/subsystem:${SUBSYSTEM}")\r
-    get_target_property(LINKER_FLAGS ${MODULE} LINK_FLAGS)\r
-    if(LINKER_FLAGS)\r
-        set(NEW_LINKER_FLAGS "${LINKER_FLAGS} ${NEW_LINKER_FLAGS}")\r
+macro(set_entrypoint MODULE ENTRYPOINT)\r
+    if(${ENTRYPOINT} STREQUAL "0")\r
+        add_linkerflag(${MODULE} "/ENTRY:0")\r
+    else()\r
+        add_linkerflag(${MODULE} "/ENTRY:${ENTRYPOINT}")\r
     endif()\r
-    set_target_properties(${MODULE} PROPERTIES LINK_FLAGS ${NEW_LINKER_FLAGS})\r
+endmacro()\r
+\r
+macro(set_subsystem MODULE SUBSYSTEM)\r
+    add_linkerflag(${MODULE} "/subsystem:${SUBSYSTEM}")\r
 endmacro()\r
 \r
 macro(set_image_base MODULE IMAGE_BASE)\r
-    set(NEW_LINKER_FLAGS "/BASE:${IMAGE_BASE}")\r
-    get_target_property(LINKER_FLAGS ${MODULE} LINK_FLAGS)\r
-    if(LINKER_FLAGS)\r
-        set(NEW_LINKER_FLAGS "${LINKER_FLAGS} ${NEW_LINKER_FLAGS}")\r
-    endif()\r
-    set_target_properties(${MODULE} PROPERTIES LINK_FLAGS ${NEW_LINKER_FLAGS})\r
+    add_linkerflag(${MODULE} "/BASE:${IMAGE_BASE}")\r
 endmacro()\r
 \r
 macro(set_module_type MODULE TYPE)\r
@@ -67,6 +60,7 @@ macro(set_module_type MODULE TYPE)
     if (${TYPE} MATCHES win32cui)\r
         set_subsystem(${MODULE} console)\r
         set_entrypoint(${MODULE} mainCRTStartup)\r
+               target_link_libraries(${MODULE} mingw_common mingw_wmain)\r
     endif ()\r
     if(${TYPE} MATCHES win32dll)\r
         # Need this only because mingw library is broken\r
@@ -77,6 +71,7 @@ macro(set_module_type MODULE TYPE)
                        message(STATUS "${MODULE} has no base address")\r
                endif()\r
                target_link_libraries(${MODULE} mingw_common mingw_dllmain)\r
+               add_importlibs(${MODULE} msvcrt kernel32)\r
     endif()\r
 \r
 endmacro()\r
@@ -122,7 +117,21 @@ macro(add_importlibs MODULE)
 endmacro()\r
 \r
 macro(pdef2def _pdef_file)\r
-# Dummy for now\r
+    get_filename_component(_file ${_pdef_file} NAME_WE)\r
+    add_custom_command(\r
+        OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/${_file}.def\r
+        COMMAND ${CMAKE_C_COMPILER} /EP /c ${CMAKE_CURRENT_SOURCE_DIR}/${_pdef_file} > ${CMAKE_CURRENT_BINARY_DIR}/${_file}.def\r
+        DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/${_pdef_file})\r
+    add_custom_target(\r
+        ${_file}_def\r
+        DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/${_file}.def)\r
+endmacro(pdef2def _pdef_file)\r
+\r
+macro(set_pdef_file _module _pdef_file)\r
+    pdef2def(${_pdef_file})\r
+    get_filename_component(_file ${_pdef_file} NAME_WE)\r
+    add_linkerflag(${_module} "/DEF:${CMAKE_CURRENT_BINARY_DIR}/${_file}.def")\r
+    add_dependencies(${_module} ${_file}_def)\r
 endmacro()\r
 \r
 file(MAKE_DIRECTORY ${CMAKE_BINARY_DIR}/importlibs)\r