Changeset 8653

Timestamp:

May 28, 2011, 10:40:10 PM (13 years ago)

Author:

rgrieder

Message:

Since we already have it now: use automatic full build units for external dependencies, even in "partial" mode.
That will not actually change anything except that Bullet is now built in a single unit instead of 3.

Location:

code/branches/unity_build

Files:

• cmake/tools/BuildUnits.cmake (modified) (3 diffs)
• cmake/tools/TargetUtilities.cmake (modified) (3 diffs)
• src/external/bullet/BulletCollision/CMakeLists.txt (modified) (2 diffs)
• src/external/bullet/BulletDynamics/CMakeLists.txt (modified) (2 diffs)
• src/external/bullet/CMakeLists.txt (modified) (1 diff)
• src/external/bullet/LinearMath/CMakeLists.txt (modified) (2 diffs)
• src/external/ogreceguirenderer/CMakeLists.txt (modified) (1 diff)
• src/external/ois/CMakeLists.txt (modified) (4 diffs)
• src/external/tinyxml/CMakeLists.txt (modified) (2 diffs)

code/branches/unity_build/cmake/tools/BuildUnits.cmake

@@ -50 +50 @@
   # However we can specify different values in a config file
   SET(_config ${BUILD_UNITS_CONFIG_${NR_OF_BUILD_UNITS}_THREADS})
+  SET(_nr_of_units)
   IF(_config)
     LIST(FIND _config ${_target_name} _index)
@@ -59 +60 @@
   ENDIF()
   IF(NOT _nr_of_units)
-    # Use default
-    SET(_nr_of_units NR_OF_BUILD_UNITS)
+    # Use default as specified (e.g. "full4" --> 4) or 1 for externals
+    IF(_arg_ORXONOX_EXTERNAL)
+      SET(_nr_of_units 1)
+    ELSE()
+      SET(_nr_of_units ${NR_OF_BUILD_UNITS})
+    ENDIF()
   ENDIF()
 
@@ -113 +118 @@
 
       # Generate the filename
-      SET(_unit_file ${CMAKE_CURRENT_BINARY_DIR}/${_target_name}BuildUnit${_unit_nr}.cc)
+      IF(NOT _nr_of_units EQUAL 1)
+        SET(_suffix ${_unit_nr})
+      ENDIF()
+      SET(_unit_file ${CMAKE_CURRENT_BINARY_DIR}/${_target_name}BuildUnit${_suffix}.cc)
       # Only write if content has changed (avoids recompile)
       IF(EXISTS ${_unit_file})

code/branches/unity_build/cmake/tools/TargetUtilities.cmake

@@ -40 +40 @@
  #      NO_INSTALL:        Do not install the target at all
  #      NO_VERSION:        Prevents adding any version to a target
+ #      NO_BUILD_UNITS:    Disables automatic (full) build units
  #
  #    Lists:
@@ -91 +92 @@
   SET(_switches   FIND_HEADER_FILES  EXCLUDE_FROM_ALL  ORXONOX_EXTERNAL
                   NO_DLL_INTERFACE   NO_SOURCE_GROUPS  PCH_NO_DEFAULT 
-                  NO_INSTALL         NO_VERSION        ${_additional_switches})
+                  NO_INSTALL         NO_VERSION        NO_BUILD_UNITS
+                  ${_additional_switches})
   SET(_list_names LINK_LIBRARIES     VERSION           SOURCE_FILES
                   DEFINE_SYMBOL      TOLUA_FILES       PCH_FILE
@@ -202 +204 @@
 
   # Full build units
-  IF(NOT _arg_ORXONOX_EXTERNAL AND ENABLE_BUILD_UNITS MATCHES "full")
-    GENERATE_BUILD_UNITS(${_target_name} _${_target_name}_files)
+  IF(ENABLE_BUILD_UNITS AND NOT _arg_NO_BUILD_UNITS)
+    # Use full build units even in partial mode for externals
+    IF(ENABLE_BUILD_UNITS MATCHES "full" OR _arg_ORXONOX_EXTERNAL)
+      GENERATE_BUILD_UNITS(${_target_name} _${_target_name}_files)
+    ENDIF()
   ENDIF()
 

code/branches/unity_build/src/external/bullet/BulletCollision/CMakeLists.txt

@@ -1 +1 @@
 ADD_SOURCE_FILES(BULLET_FILES
-
-BUILD_UNIT BulletCollisionBuildUnit.cpp
 
         BroadphaseCollision/btAxisSweep3.cpp
@@ -84 +82 @@
         NarrowPhaseCollision/btMinkowskiPenetrationDepthSolver.cpp
         NarrowPhaseCollision/btPersistentManifold.cpp
+        NarrowPhaseCollision/btPolyhedralContactClipping.cpp
         NarrowPhaseCollision/btRaycastCallback.cpp
         NarrowPhaseCollision/btSubSimplexConvexCast.cpp
         NarrowPhaseCollision/btVoronoiSimplexSolver.cpp
 
-END_BUILD_UNIT
-
-    # Raises compiler errors when compiled inside the build unit
-        NarrowPhaseCollision/btPolyhedralContactClipping.cpp
 
         # Headers

code/branches/unity_build/src/external/bullet/BulletDynamics/CMakeLists.txt

@@ -1 +1 @@
 ADD_SOURCE_FILES(BULLET_FILES
-
-BUILD_UNIT BulletDynamicsBuildUnit.cpp
 
         Character/btKinematicCharacterController.cpp
@@ -25 +23 @@
         Vehicle/btRaycastVehicle.cpp
         Vehicle/btWheelInfo.cpp
-
-END_BUILD_UNIT
 
         # Headers

code/branches/unity_build/src/external/bullet/CMakeLists.txt

@@ -32 +32 @@
   VERSION
     2.78
+  EXCLUDE_FROM_BUILD_UNITS
+    BulletCollision/NarrowPhaseCollision/btPolyhedralContactClipping.cpp
   SOURCE_FILES
     ${BULLET_FILES}

code/branches/unity_build/src/external/bullet/LinearMath/CMakeLists.txt

@@ -1 +1 @@
 ADD_SOURCE_FILES(BULLET_FILES
 
-BUILD_UNIT BulletLinearMathBuildUnit.cpp
         btAlignedAllocator.cpp
         btConvexHull.cpp
@@ -8 +7 @@
         btQuickprof.cpp
         btSerializer.cpp
-END_BUILD_UNIT
 
         # Headers

code/branches/unity_build/src/external/ogreceguirenderer/CMakeLists.txt

@@ -23 +23 @@
   OgreCEGUITexture.h
 
-BUILD_UNIT OgreCEGUIRendererBuildUnit.cpp
   OgreCEGUIRenderer.cpp
   OgreCEGUIResourceProvider.cpp
   OgreCEGUITexture.cpp
-END_BUILD_UNIT
 )
 

code/branches/unity_build/src/external/ois/CMakeLists.txt

@@ -34 +34 @@
   OISObject.h
   OISPrereqs.h
-)
 
-# Put everything into one single build unit (doens't work on OS X though)
-IF(NOT APPLE)
-  ADD_SOURCE_FILES(OIS_FILES BUILD_UNIT OISBuildUnit.cpp)
-ENDIF()
-
-ADD_SOURCE_FILES(OIS_FILES
   OISEffect.cpp
   OISException.cpp
@@ -50 +43 @@
   OISObject.cpp
 )
+
 IF(WIN32)
   ADD_SUBDIRECTORY(win32)
@@ -58 +52 @@
 ENDIF()
 
-# Close build unit from above
-IF(NOT APPLE)
-  ADD_SOURCE_FILES(OIS_FILES END_BUILD_UNIT)
-ENDIF()
-
 # MinGW doesn't come with some required Windows headers
 IF(MINGW)
   INCLUDE_DIRECTORIES(${WMI_INCLUDE_DIR})
+ENDIF()
+
+# Apple has problems with OIS and build units
+IF(APPLE)
+  SET(USE_BUILD_UNITS NO_BUILD_UNITS)
 ENDIF()
 
@@ -82 +76 @@
   LINK_LIBS_LINUX
     X11
+  ${USE_BUILD_UNITS}
   SOURCE_FILES
     ${OIS_FILES}

code/branches/unity_build/src/external/tinyxml/CMakeLists.txt

@@ -24 +24 @@
   tinyxml.h
 
-BUILD_UNIT TicppBuildUnit.cpp
   ticpp.cpp
   tinystr.cpp
@@ -30 +29 @@
   tinyxmlerror.cpp
   tinyxmlparser.cpp
-END_BUILD_UNIT
 )