Changeset 7138

Timestamp:

Jun 10, 2010, 1:00:59 AM (14 years ago)

Author:

rgrieder

Message:

DEFINE_SYMBOL should now work for static linking as well.

File:

• code/branches/presentation3/cmake/TargetUtilities.cmake (modified) (1 diff)

code/branches/presentation3/cmake/TargetUtilities.cmake

@@ -230 +230 @@
 
   # DEFINE_SYMBOL
-  IF(_arg_DEFINE_SYMBOL)
-    # Format is: static "static_symbol" shared "shared_symbol"
-    # but the order doesn't matter
-    LIST(LENGTH _arg_DEFINE_SYMBOL _define_symbol_length)
-    IF (_define_symbol_length LESS 2)
-      MESSAGE(FATAL_ERROR "Number of expected arguments for DEFINE_SYMBOL is at least 2: static \"STATIC_SYMBOL\" shared \"SHARED_SYMBOL\"")
-    ENDIF()
-    STRING(TOLOWER "${_arg_STATIC}${_arg_SHARED}" _static_shared_lower)
-    LIST(FIND _arg_DEFINE_SYMBOL ${_static_shared_lower} _symbol_definition_index)
-    MATH(EXPR _symbol_definition_index "${_symbol_definition_index} + 1")
-    IF(_symbol_definition_index LESS _define_symbol_length)
-      LIST(GET _arg_DEFINE_SYMBOL ${_symbol_definition_index} _symbol_definition)
+  IF(_arg_DEFINE_SYMBOL OR NOT _arg_ORXONOX_EXTERNAL)
+    IF(_arg_DEFINE_SYMBOL)
+      # Format is: static "static_symbol" shared "shared_symbol"
+      # but the order doesn't matter
+      LIST(LENGTH _arg_DEFINE_SYMBOL _define_symbol_length)
+      IF (_define_symbol_length LESS 2)
+        MESSAGE(FATAL_ERROR "Number of expected arguments for DEFINE_SYMBOL is at least 2: static \"STATIC_SYMBOL\" shared \"SHARED_SYMBOL\"")
+      ENDIF()
+      STRING(TOLOWER "${_arg_STATIC}${_arg_SHARED}" _static_shared_lower)
+      LIST(FIND _arg_DEFINE_SYMBOL ${_static_shared_lower} _symbol_definition_index)
+      MATH(EXPR _symbol_definition_index "${_symbol_definition_index} + 1")
+      IF(_symbol_definition_index LESS _define_symbol_length)
+        LIST(GET _arg_DEFINE_SYMBOL ${_symbol_definition_index} _symbol_definition)
+      ENDIF()
+    ELSE()
+      # Automatically add the macro definitions for our own libraries
+      SET(_symbol_definition "${_target_name_upper}_${_arg_STATIC}${_arg_SHARED}_BUILD")
+    ENDIF()
+
+    # Use the DEFINE_SYMBOL property for shared builds (not used by CMake for static builds),
+    # but if we only use COMPILE_FLAGS, CMake will define a symbol anyway.
+    IF(_arg_SHARED)
       SET_TARGET_PROPERTIES(${_target_name} PROPERTIES DEFINE_SYMBOL ${_symbol_definition})
-    ENDIF()
-  ELSEIF(NOT _arg_ORXONOX_EXTERNAL)
-    # Automatically add the macro definitions for our own libraries
-    SET_TARGET_PROPERTIES(${_target_name} PROPERTIES DEFINE_SYMBOL "${_target_name_upper}_${_arg_STATIC}${_arg_SHARED}_BUILD")
+    ELSE()
+      GET_TARGET_PROPERTY(_compile_flags ${_target_name} COMPILE_FLAGS)
+      IF(NOT _compile_flags)
+        SET(_compile_flags)
+      ENDIF()
+      SET_TARGET_PROPERTIES(${_target_name} PROPERTIES COMPILE_FLAGS "${_compile_flags} -D${_symbol_definition}")
+    ENDIF()
   ENDIF()