cmake: objects for AD, MD, SS c-bind shared/static lib builds

Without this approach, errors occur when making the _C_Binding.f90 for static and shared in parallel.

Author: andrew-platt

modules/aerodyn/CMakeLists.txt

@@ -98,23 +98,30 @@ add_executable(unsteadyaero_driver
 )
 target_link_libraries(unsteadyaero_driver basicaerolib lindynlib versioninfolib)
 
+
 # AeroDyn-InflowWind c-bindings interface library
-add_library(aerodyn_inflow_c_binding SHARED 
+#   create object instead of directly linking into shared and static -- causes issues in parallel builds
+#   This is only required because we are static linking the library for wavetank
+#   NOTE: target linking at the object, static, and shared libraries.  Different CMake versions handle this
+#         slightly differently with unpredictable results if I don't.
+add_library(aerodyn_inflow_c_binding_object OBJECT
   src/AeroDyn_Inflow_C_Binding_Types.f90
   src/AeroDyn_Inflow_C_Binding.f90
 )
-target_link_libraries(aerodyn_inflow_c_binding aerodyn_driver_subs versioninfolib)
+target_link_libraries(aerodyn_inflow_c_binding_object adilib aerodyn_driver_subs nwtclibs versioninfolib)
+set_property(TARGET aerodyn_inflow_c_binding_object PROPERTY POSITION_INDEPENDENT_CODE 1) # required for shared libs
+
+#   shared
+add_library(aerodyn_inflow_c_binding SHARED $<TARGET_OBJECTS:aerodyn_inflow_c_binding_object>)
+target_link_libraries(aerodyn_inflow_c_binding adilib aerodyn_driver_subs nwtclibs versioninfolib)
 if(APPLE OR UNIX)
    target_compile_definitions(aerodyn_inflow_c_binding PRIVATE IMPLICIT_DLLEXPORT)
 endif()
 
 # C-bindings non-shared interface
 # This is a workaround for building wavetank into a single DLL (also allows setting CU globaly for sending screen to file for labview integration)
-add_library(aerodyn_inflow_c_bind_static STATIC
-  src/AeroDyn_Inflow_C_Binding_Types.f90
-  src/AeroDyn_Inflow_C_Binding.f90
-)
-target_link_libraries(aerodyn_inflow_c_bind_static aerodyn_driver_subs versioninfolib)
+add_library(aerodyn_inflow_c_bind_static STATIC $<TARGET_OBJECTS:aerodyn_inflow_c_binding_object>)
+target_link_libraries(aerodyn_inflow_c_bind_static adilib aerodyn_driver_subs nwtclibs versioninfolib)
 if(APPLE OR UNIX)
    target_compile_definitions(aerodyn_inflow_c_bind_static PRIVATE IMPLICIT_DLLEXPORT)
 endif()

modules/moordyn/CMakeLists.txt

@@ -37,20 +37,25 @@ add_executable(moordyn_driver
 target_link_libraries(moordyn_driver moordynlib versioninfolib)
 
 # C-bindings interface library
-add_library(moordyn_c_binding SHARED 
-  src/MoorDyn_C_Binding.f90
-)
-target_link_libraries(moordyn_c_binding moordynlib seastlib versioninfolib)
+#   create object instead of directly linking into shared and static -- causes issues in parallel builds
+#   This is only required because we are static linking the library for wavetank
+#   NOTE: target linking at the object, static, and shared libraries.  Different CMake versions handle this
+#         slightly differently with unpredictable results if I don't.
+add_library(moordyn_c_binding_object OBJECT src/MoorDyn_C_Binding.f90)
+target_link_libraries(moordyn_c_binding_object moordynlib seastlib nwtclibs versioninfolib)
+set_property(TARGET moordyn_c_binding_object PROPERTY POSITION_INDEPENDENT_CODE 1) # required for shared libs
+
+#   shared
+add_library(moordyn_c_binding SHARED $<TARGET_OBJECTS:moordyn_c_binding_object>)
+target_link_libraries(moordyn_c_binding moordynlib seastlib nwtclibs versioninfolib)
 if(APPLE OR UNIX)
    target_compile_definitions(moordyn_c_binding PRIVATE IMPLICIT_DLLEXPORT)
 endif()
 
 # C-bindings non-shared interface
 # This is a workaround for building wavetank into a single DLL (also allows setting CU globaly for sending screen to file for labview integration)
-add_library(moordyn_c_bind_static STATIC 
-  src/MoorDyn_C_Binding.f90
-)
-target_link_libraries(moordyn_c_bind_static moordynlib seastlib versioninfolib)
+add_library(moordyn_c_bind_static SHARED $<TARGET_OBJECTS:moordyn_c_binding_object>)
+target_link_libraries(moordyn_c_bind_static moordynlib seastlib nwtclibs versioninfolib)
 if(APPLE OR UNIX)
    target_compile_definitions(moordyn_c_bind_static PRIVATE IMPLICIT_DLLEXPORT)
 endif()

modules/seastate/CMakeLists.txt

@@ -39,21 +39,27 @@ add_library(seastlib STATIC
 )
 target_link_libraries(seastlib nwtclibs versioninfolib)
 
+
 # C-bindings interface library
-add_library(seastate_c_binding SHARED 
-  src/SeaState_C_Binding.f90
-)
-target_link_libraries(seastate_c_binding seastlib versioninfolib)
+#   create object instead of directly linking into shared and static -- causes issues in parallel builds
+#   This is only required because we are static linking the library for wavetank
+#   NOTE: target linking at the object, static, and shared libraries.  Different CMake versions handle this
+#         slightly differently with unpredictable results if I don't.
+add_library(seastate_c_binding_object OBJECT src/SeaState_C_Binding.f90)
+target_link_libraries(seastate_c_binding_object seastlib nwtclibs versioninfolib)
+set_property(TARGET seastate_c_binding_object PROPERTY POSITION_INDEPENDENT_CODE 1) # required for shared libs
+
+#   shared
+add_library(seastate_c_binding SHARED $<TARGET_OBJECTS:seastate_c_binding_object>)
+target_link_libraries(seastate_c_binding seastlib nwtclibs versioninfolib)
 if(APPLE OR UNIX)
   target_compile_definitions(seastate_c_binding PRIVATE IMPLICIT_DLLEXPORT)
 endif()
 
-# C-bindings non-shared interface
+# C-bindings static interface
 # This is a workaround for building wavetank into a single DLL (also allows setting CU globaly for sending screen to file for labview integration)
-add_library(seastate_c_bind_static STATIC
-  src/SeaState_C_Binding.f90
-)
-target_link_libraries(seastate_c_bind_static seastlib versioninfolib)
+add_library(seastate_c_bind_static STATIC $<TARGET_OBJECTS:seastate_c_binding_object>)
+target_link_libraries(seastate_c_bind_static seastlib nwtclibs versioninfolib)
 if(APPLE OR UNIX)
   target_compile_definitions(seastate_c_bind_static PRIVATE IMPLICIT_DLLEXPORT)
 endif()