Changeset 5649 for code/branches/libraries

Timestamp:

Aug 16, 2009, 4:21:01 PM (15 years ago)

Author:

rgrieder

Message:

Fixed msvc build: I had to link the plugins as SHARED instead of MODULE because CMake doesn't allow modules to be linked against another module.
The reason why it did work after all was because the quest library was built before overlays causing CMake to link against whatever it found under the name "overlays" in the end (which was of course the output file instead of the CMake representation of a library). Since msvc never links against dlls but rather against the corresponding *.lib file things got completely busted.

Also added the proposed fix for the PATH variable (successfully tested under Windows by deleting the overlays.plugin file).

Location:

code/branches/libraries

Files:

• cmake/TargetUtilities.cmake (modified) (3 diffs)
• src/core/Core.cc (modified) (1 diff)
• src/orxonox/CMakeLists.txt (modified) (1 diff)

code/branches/libraries/cmake/TargetUtilities.cmake

@@ -148 +148 @@
   ENDIF()
 
-  # PLUGIN A
+  # PLUGIN A, always create shared libraries
   IF(_arg_PLUGIN)
-    SET(_arg_SHARED MODULE)
+    SET(_arg_SHARED SHARED)
     SET(_arg_STATIC)
   ENDIF()
@@ -165 +165 @@
   # PLUGIN B
   IF (_arg_PLUGIN)
-    SET_TARGET_PROPERTIES(${_target_name} PROPERTIES LIBRARY_OUTPUT_DIRECTORY ${CMAKE_PLUGIN_OUTPUT_DIRECTORY})
+    SET_TARGET_PROPERTIES(${_target_name} PROPERTIES
+      RUNTIME_OUTPUT_DIRECTORY ${CMAKE_PLUGIN_OUTPUT_DIRECTORY} # Windows
+      LIBRARY_OUTPUT_DIRECTORY ${CMAKE_PLUGIN_OUTPUT_DIRECTORY} # Unix
+    )
     ADD_PLUGIN(${_target_name})
   ENDIF()
@@ -197 +200 @@
 
   IF(NOT _arg_STATIC AND NOT _arg_NO_INSTALL)
-    SET(_library_destination ${ORXONOX_LIBRARY_INSTALL_PATH})
-    IF (_arg_PLUGIN)
-      SET(_library_destination ${ORXONOX_PLUGIN_INSTALL_PATH})
-    ENDIF()
-
-    INSTALL(TARGETS ${_target_name}
-      RUNTIME DESTINATION ${ORXONOX_RUNTIME_INSTALL_PATH}
-      LIBRARY DESTINATION ${_library_destination}
-    )
+    IF(_arg_PLUGIN)
+      INSTALL(TARGETS ${_target_name}
+        RUNTIME DESTINATION ${ORXONOX_PLUGIN_INSTALL_PATH}
+        LIBRARY DESTINATION ${ORXONOX_PLUGIN_INSTALL_PATH}
+      )
+    ELSE()
+      INSTALL(TARGETS ${_target_name}
+        RUNTIME DESTINATION ${ORXONOX_RUNTIME_INSTALL_PATH}
+        LIBRARY DESTINATION ${ORXONOX_LIBRARY_INSTALL_PATH}
+      )
+    ENDIF()
   ENDIF()
 

code/branches/libraries/src/core/Core.cc

@@ -282 +282 @@
             boost::filesystem::path searchpath = this->configuration_->pluginPath_;
 
+            // Add that path to the PATH variable in case a plugin depends on another one
+            std::string pathVariable = getenv("PATH");
+            putenv(("PATH=" + pathVariable + ";" + configuration_->pluginPath_.string()).c_str());
+
             boost::filesystem::directory_iterator file(searchpath);
             boost::filesystem::directory_iterator end;

code/branches/libraries/src/orxonox/CMakeLists.txt

@@ -31 +31 @@
 ADD_SUBDIRECTORY(gamestates)
 ADD_SUBDIRECTORY(interfaces)
+ADD_SUBDIRECTORY(overlays)
 ADD_SUBDIRECTORY(objects)
-ADD_SUBDIRECTORY(overlays)
 ADD_SUBDIRECTORY(sound)
 ADD_SUBDIRECTORY(tools)