Set CMAKE_SHARED_LIBRARY_SUFFIX to always create .so files (#207) (#257)

* Set CMAKE_SHARED_LIBRARY_SUFFIX to always create .so files.

* [#23543] Added cmake option and default suffix for platforms



* [#23543] Added cmake option for Python test and examples



* [#23543] Default value for Windows OS



* [#23543] Review changes



* Refs #23543. Regenerate code with Fast DDS Gen from related PR.



---------






(cherry picked from commit 57a5541)

Signed-off-by: danipiza <dpizarrogallego@gmail.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Co-authored-by: Keane Dixon Wong <Keanewong913@gmail.com>
Co-authored-by: Keane Wong <kwong@FELT241.FluxErgy.local>
Co-authored-by: danipiza <dpizarrogallego@gmail.com>
Co-authored-by: Miguel Company <MiguelCompany@eprosima.com>

Author: 5 people

fastdds_python/CMakeLists.txt

@@ -26,6 +26,13 @@ endif()
 
 project(fastdds_python VERSION 2.3.0)
 
+if(NOT WIN32)
+    # Default values for shared library suffix in MacOS
+    if(${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
+        set(CMAKE_SHARED_LIBRARY_SUFFIX ".so")
+    endif()
+endif()
+
 # Set BUILD_TESTING to OFF by default.
 if(NOT BUILD_TESTING)
     message(STATUS "Tests not compiled by default")

fastdds_python/test/types/CMakeLists.txt

@@ -41,6 +41,13 @@ find_package(fastdds 3 REQUIRED)
 
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
 
+if(NOT WIN32)
+    # Default values for shared library suffix in MacOS
+    if(${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
+        set(CMAKE_SHARED_LIBRARY_SUFFIX ".so")
+    endif()
+endif()
+
 #Create library for C++ types
 add_library(${PROJECT_NAME} SHARED
         test_included_modulesTypeObjectSupport.cxx
@@ -146,6 +153,13 @@ find_package(fastdds 3 REQUIRED)
 
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
 
+if(NOT WIN32)
+    # Default values for shared library suffix in MacOS
+    if(${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
+        set(CMAKE_SHARED_LIBRARY_SUFFIX ".so")
+    endif()
+endif()
+
 #Create library for C++ types
 add_library(${PROJECT_NAME} SHARED
         test_modulesTypeObjectSupport.cxx
@@ -251,6 +265,13 @@ find_package(fastdds 3 REQUIRED)
 
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
 
+if(NOT WIN32)
+    # Default values for shared library suffix in MacOS
+    if(${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
+        set(CMAKE_SHARED_LIBRARY_SUFFIX ".so")
+    endif()
+endif()
+
 #Create library for C++ types
 add_library(${PROJECT_NAME} SHARED
         test_completeTypeObjectSupport.cxx

fastdds_python/test/types/test_complete.i

@@ -233,6 +233,11 @@ namespace swig {
         eprosima::fastcdr::fixed_string<16> tmp(value);
         $self->fixed_string_field(tmp);
     }
+
+    std::string fixed_string_field_str() const
+    {
+        return std::string($self->fixed_string_field(), strnlen($self->fixed_string_field(), 16));
+    }
 }
 
 
@@ -444,6 +449,11 @@ namespace swig {
         eprosima::fastcdr::fixed_string<16> tmp(value);
         $self->fixed_string_field(tmp);
     }
+
+    std::string fixed_string_field_str() const
+    {
+        return std::string($self->fixed_string_field(), strnlen($self->fixed_string_field(), 16));
+    }
 }
 
 
@@ -1699,6 +1709,11 @@ namespace swig {
         eprosima::fastcdr::fixed_string<16> tmp(value);
         $self->fixed_string_field(tmp);
     }
+
+    std::string fixed_string_field_str() const
+    {
+        return std::string($self->fixed_string_field(), strnlen($self->fixed_string_field(), 16));
+    }
 }
 
 

fastdds_python/test/types/test_modules.i

@@ -232,6 +232,11 @@ namespace swig {
         eprosima::fastcdr::fixed_string<16> tmp(value);
         $self->fixed_string_field(tmp);
     }
+
+    std::string fixed_string_field_str() const
+    {
+        return std::string($self->fixed_string_field(), strnlen($self->fixed_string_field(), 16));
+    }
 }
 
 
@@ -432,6 +437,11 @@ namespace swig {
         eprosima::fastcdr::fixed_string<16> tmp(value);
         $self->fixed_string_field(tmp);
     }
+
+    std::string fixed_string_field_str() const
+    {
+        return std::string($self->fixed_string_field(), strnlen($self->fixed_string_field(), 16));
+    }
 }
 
 
@@ -1687,6 +1697,11 @@ namespace swig {
         eprosima::fastcdr::fixed_string<16> tmp(value);
         $self->fixed_string_field(tmp);
     }
+
+    std::string fixed_string_field_str() const
+    {
+        return std::string($self->fixed_string_field(), strnlen($self->fixed_string_field(), 16));
+    }
 }
 
 

fastdds_python_examples/HelloWorldExample/generated_code/CMakeLists.txt

@@ -41,6 +41,13 @@ find_package(fastdds 3 REQUIRED)
 
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
 
+if(NOT WIN32)
+    # Default values for shared library suffix in MacOS
+    if(${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
+        set(CMAKE_SHARED_LIBRARY_SUFFIX ".so")
+    endif()
+endif()
+
 #Create library for C++ types
 add_library(${PROJECT_NAME} SHARED
         HelloWorldTypeObjectSupport.cxx

fastdds_python_examples/RPCExample/generated_code/CMakeLists.txt

@@ -41,6 +41,13 @@ find_package(fastdds 3 REQUIRED)
 
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
 
+if(NOT WIN32)
+    # Default values for shared library suffix in MacOS
+    if(${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
+        set(CMAKE_SHARED_LIBRARY_SUFFIX ".so")
+    endif()
+endif()
+
 #Create library for C++ types
 add_library(${PROJECT_NAME} SHARED
         calculatorTypeObjectSupport.cxx