search

robotology / yarp-devices-ros2

Plugins, devices and network wrappers for Yarp-ROS2 interoperation
http://www.yarp.it
10 stars 9 forks source link

Add conda-based CI for Windows, macOS and Linux #11

Closed traversaro closed 3 years ago

traversaro commented 3 years ago

Just a WIP, to test the newly available ROS galactic packages being worked on at https://github.com/RoboStack/ros-galactic/ .

traversaro commented 3 years ago

The CI is now failing with:

[43/52] Building CXX object src/devices/map2D_nws_ros2/CMakeFiles/yarp_map2D_nws_ros2.dir/yarp_plugin_map2D_nws_ros2.cpp.o
FAILED: src/devices/map2D_nws_ros2/CMakeFiles/yarp_map2D_nws_ros2.dir/yarp_plugin_map2D_nws_ros2.cpp.o 
/usr/share/miniconda/envs/test/bin/x86_64-conda-linux-gnu-c++ -DDEFAULT_RMW_IMPLEMENTATION=rmw_cyclonedds_cpp -Dyarp_map2D_nws_ros2_EXPORTS -I/home/runner/work/yarp-ros2/yarp-ros2/build/ros2_interfaces_ws/src/map2d_nws_ros2_msgs/rosidl_generator_cpp -Wall -Wextra -Wsign-compare -Wpointer-arith -Winit-self -Wnon-virtual-dtor -Wcast-align -Wunused -Wunused-but-set-variable -Wvla -Wmissing-include-dirs -Wlogical-op -Wreorder -Wsizeof-pointer-memaccess -Woverloaded-virtual -Wundef -Wredundant-decls -Wunknown-pragmas -Wunused-result -Wc++17-compat -Wignored-attributes -Wdangling-else -Wmisleading-indentation -Wtautological-compare -Wsuggest-override -Wmaybe-uninitialized  -Wno-unused-parameter  -Wdeprecated-declarations -fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem /usr/share/miniconda/envs/test/include -fvisibility-inlines-hidden -std=c++17 -fmessage-length=0 -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem /usr/share/miniconda/envs/test/include -O3 -DNDEBUG -fPIC -MD -MT src/devices/map2D_nws_ros2/CMakeFiles/yarp_map2D_nws_ros2.dir/yarp_plugin_map2D_nws_ros2.cpp.o -MF src/devices/map2D_nws_ros2/CMakeFiles/yarp_map2D_nws_ros2.dir/yarp_plugin_map2D_nws_ros2.cpp.o.d -o src/devices/map2D_nws_ros2/CMakeFiles/yarp_map2D_nws_ros2.dir/yarp_plugin_map2D_nws_ros2.cpp.o -c /home/runner/work/yarp-ros2/yarp-ros2/build/src/devices/map2D_nws_ros2/yarp_plugin_map2D_nws_ros2.cpp
In file included from /home/runner/work/yarp-ros2/yarp-ros2/build/src/devices/map2D_nws_ros2/yarp_plugin_map2D_nws_ros2.cpp:10:
/home/runner/work/yarp-ros2/yarp-ros2/src/devices/map2D_nws_ros2/map2D_nws_ros2.h:53:10: fatal error: nav_msgs/msg/rosidl_typesupport_connext_cpp__visibility_control.h: No such file or directory
   53 | #include <nav_msgs/msg/rosidl_typesupport_connext_cpp__visibility_control.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

It seems a foxy vs galactic kind of change, as in galactic connext was substiutied by connextdds see https://github.com/ros2/rmw_connext#rmw_connext .

traversaro commented 3 years ago

I removed all the visibility headers inclusion, and everything is working fine, so I assume that is ok and the inclusion were unnecessary. Now the test are instead failing with error: 

3/3 Test #3: xmllint ..........................***Failed    0.33 sec
-- run_test.py: invoking following command in '/home/runner/work/yarp-ros2/yarp-ros2/ros2_interfaces_ws/src/map2d_nws_ros2_msgs':
 - /usr/share/miniconda/envs/test/bin/ament_xmllint --xunit-file /home/runner/work/yarp-ros2/yarp-ros2/build/test_results/map2d_nws_ros2_msgs/xmllint.xunit.xml
Errors while running CTest
File 'package.xml' is invalid:
/home/runner/work/yarp-ros2/yarp-ros2/ros2_interfaces_ws/src/map2d_nws_ros2_msgs/package.xml:20: element test_depend: Schemas validity error : Element 'test_depend': This element is not expected. Expected is one of ( member_of_group, export ).
/home/runner/work/yarp-ros2/yarp-ros2/ros2_interfaces_ws/src/map2d_nws_ros2_msgs/package.xml fails to validate

1 files are invalid
-- run_test.py: return code 1
-- run_test.py: verify result file '/home/runner/work/yarp-ros2/yarp-ros2/build/test_results/map2d_nws_ros2_msgs/xmllint.xunit.xml'

67% tests passed, 1 tests failed out of 3

Label Time Summary:
copyright     =   0.17 sec*proc (1 test)
lint_cmake    =   0.13 sec*proc (1 test)
linter        =   0.64 sec*proc (3 tests)
xmllint       =   0.33 sec*proc (1 test)

Total Test time (real) =   0.64 sec

The following tests FAILED:
      3 - xmllint (Failed)
Error: Process completed with exit code 8.
ste93 commented 3 years ago

I removed all the visibility headers inclusion, and everything is working fine, so I assume that is ok and the inclusion were unnecessary. Now the test are instead failing with error:

3/3 Test #3: xmllint ..........................***Failed    0.33 sec
-- run_test.py: invoking following command in '/home/runner/work/yarp-ros2/yarp-ros2/ros2_interfaces_ws/src/map2d_nws_ros2_msgs':
 - /usr/share/miniconda/envs/test/bin/ament_xmllint --xunit-file /home/runner/work/yarp-ros2/yarp-ros2/build/test_results/map2d_nws_ros2_msgs/xmllint.xunit.xml
Errors while running CTest
File 'package.xml' is invalid:
/home/runner/work/yarp-ros2/yarp-ros2/ros2_interfaces_ws/src/map2d_nws_ros2_msgs/package.xml:20: element test_depend: Schemas validity error : Element 'test_depend': This element is not expected. Expected is one of ( member_of_group, export ).
/home/runner/work/yarp-ros2/yarp-ros2/ros2_interfaces_ws/src/map2d_nws_ros2_msgs/package.xml fails to validate

1 files are invalid
-- run_test.py: return code 1
-- run_test.py: verify result file '/home/runner/work/yarp-ros2/yarp-ros2/build/test_results/map2d_nws_ros2_msgs/xmllint.xunit.xml'

67% tests passed, 1 tests failed out of 3

Label Time Summary:
copyright     =   0.17 sec*proc (1 test)
lint_cmake    =   0.13 sec*proc (1 test)
linter        =   0.64 sec*proc (3 tests)
xmllint       =   0.33 sec*proc (1 test)

Total Test time (real) =   0.64 sec

The following tests FAILED:
    3 - xmllint (Failed)
Error: Process completed with exit code 8.

I've done some tests now and it seems that this happens only in galactic and not in foxy (until now we are testing it on foxy and not in galactic) I think that it is a problem that must be fixed in the passage. I will go deep and check how can it be solved. Thank you for raising the problem ;)

ste93 commented 3 years ago

You just need to install ros-galactic-test-msgs and it should be fine, i'll add the dependency in the readme.

ste93 commented 3 years ago

sorry @traversaro I was correcting the wrong mistake, the problem above can be solved by editing ros2_interfaces_ws/src/map2d_nws_ros2_msgs/package.xml and move line rosidl_interface_packages below the two test_depend lines.

traversaro commented 3 years ago

sorry @traversaro I was correcting the wrong mistake, the problem above can be solved by editing ros2_interfaces_ws/src/map2d_nws_ros2_msgs/package.xml and move line rosidl_interface_packages below the two test_depend lines.

Cool! Do you want to to open a PR just for that change? Otherwise feel free to push directly to this branch, I guess you should have the permissions.

traversaro commented 3 years ago

Linux works finally fine after https://github.com/robotology-playground/yarp-ros2/pull/13 . macOS is failing with:

dyld: Library not loaded: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
  Referenced from: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/vecLib
  Reason: Incompatible library version: vecLib requires version 1.0.0 or later, but libBLAS.dylib provides version 0.0.0
/Users/runner/work/_temp/ffb19b6f-6f5a-4665-a4c3-1e96c8c86f4a.sh: line 4: 27867 Abort trap: 6           cmake -GNinja -DBUILD_TESTING:BOOL=ON -DYARP_ROS2_USE_SYSTEM_map2d_nws_ros2_msgs:BOOL=OFF -DCMAKE_BUILD_TYPE=Release ..

I think it is a known problem for which there is a workaround, I need to check it.

traversaro commented 3 years ago

Linux works finally fine after #13 . macOS is failing with:

dyld: Library not loaded: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
  Referenced from: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/vecLib
  Reason: Incompatible library version: vecLib requires version 1.0.0 or later, but libBLAS.dylib provides version 0.0.0
/Users/runner/work/_temp/ffb19b6f-6f5a-4665-a4c3-1e96c8c86f4a.sh: line 4: 27867 Abort trap: 6           cmake -GNinja -DBUILD_TESTING:BOOL=ON -DYARP_ROS2_USE_SYSTEM_map2d_nws_ros2_msgs:BOOL=OFF -DCMAKE_BUILD_TYPE=Release ..

I think it is a known problem for which there is a workaround, I need to check it.

Found it: https://github.com/RoboStack/ros-galactic/issues/39 . Probably for the time being we can just unset DYLD_LIBRARY_PATH .

traversaro commented 3 years ago

Windows is instead failing with:

-- The CXX compiler identification is MSVC 19.29.30133.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.29.30133/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found YCM: C:/Miniconda/envs/test/Library/share/cmake/YCM (found version "0.13.0")
-- Found YARP: C:/Miniconda/envs/test/Library/lib/cmake/YARP (found version "3.5.0+108-20210824.10+git218727fbf")
-- Found ament_cmake: 1.1.4 (C:/Miniconda/envs/test/Library/share/ament_cmake/cmake)
-- Found PythonInterp: C:/hostedtoolcache/windows/Python/3.7.9/x64/python3.exe (found suitable version "3.7.9", minimum required is "3") 
-- Using PYTHON_EXECUTABLE: C:/hostedtoolcache/windows/Python/3.7.9/x64/python3.exe
Traceback (most recent call last):
  File "C:/Miniconda/envs/test/Library/share/ament_cmake_core/cmake/package_templates/templates_2_cmake.py", line 21, in <module>
    from ament_package.templates import get_environment_hook_template_path
ModuleNotFoundError: No module named 'ament_package'
CMake Error at C:/Miniconda/envs/test/Library/share/ament_cmake_core/cmake/ament_cmake_package_templates-extras.cmake:41 (message):
  execute_process(C:/hostedtoolcache/windows/Python/3.7.9/x64/python3.exe
  C:/Miniconda/envs/test/Library/share/ament_cmake_core/cmake/package_templates/templates_2_cmake.py
-- Configuring incomplete, errors occurred!
See also "D:/a/yarp-ros2/yarp-ros2/build/CMakeFiles/CMakeOutput.log".
  D:/a/yarp-ros2/yarp-ros2/build/ament_cmake_package_templates/templates.cmake)
  returned error code 1
Call Stack (most recent call first):
  C:/Miniconda/envs/test/Library/share/ament_cmake_core/cmake/ament_cmake_coreConfig.cmake:41 (include)
  C:/Miniconda/envs/test/Library/share/ament_cmake/cmake/ament_cmake_export_dependencies-extras.cmake:15 (find_package)
  C:/Miniconda/envs/test/Library/share/ament_cmake/cmake/ament_cmakeConfig.cmake:41 (include)
  CMakeLists.txt:57 (find_package)

It seems that for some reason the wrong Python is found, I wonder if this is related to https://github.com/ros2/python_cmake_module/issues/6 or one of the related PRs (the one cross-linked to the issue).

drdanz commented 3 years ago

I imported the changes to src/devices/map2D_nws_ros2/map2D_nws_ros2.h in master, since it fixes the build... See 20fa70d4305d738c9611b1886489d0fabc96d082 Please rebase before merging this

traversaro commented 3 years ago

I imported the changes to src/devices/map2D_nws_ros2/map2D_nws_ros2.h in master, since it fixes the build... See 20fa70d Please rebase before merging this

Ack, thanks!

ste93 commented 3 years ago

Found it: RoboStack/ros-galactic#39 . Probably for the time being we can just unset DYLD_LIBRARY_PATH .

I think unset it can be a good solution for now.

traversaro commented 3 years ago

After some additional tweaks (mainly due to the fact that the ROS2 activation scripts on Windows do not work on Bash) both macOS and Windows are failing with a linker error:

[35/52] Linking CXX shared module lib/yarp/yarp_ros2test.so
FAILED: lib/yarp/yarp_ros2test.so 
: && /usr/local/miniconda/envs/test/bin/x86_64-apple-darwin13.4.0-clang++ -Wall -Wextra -Wsign-compare -Wpointer-arith -Winit-self -Wnon-virtual-dtor -Wcast-align -Wunused -Wvla -Wmissing-include-dirs -Wreorder -Wsizeof-pointer-memaccess -Woverloaded-virtual -Wtautological-undefined-compare -Wmismatched-new-delete -Wparentheses-equality -Wundef -Wredundant-decls -Wunknown-pragmas -Wunused-result -Wc++17-compat -Wc++2a-compat -Wheader-guard -Wignored-attributes -Wnewline-eof -Wdangling-else -Wgcc-compat -Wmicrosoft-exists -Wstatic-inline-explicit-instantiation -Wmisleading-indentation -Wtautological-compare -Winconsistent-missing-override -Wsuggest-override -Wnull-conversion  -Wno-unused-parameter  -Wdeprecated-declarations -march=core2 -mtune=haswell -mssse3 -ftree-vectorize -fPIC -fPIE -fstack-protector-strong -O2 -pipe -stdlib=libc++ -fvisibility-inlines-hidden -std=c++14 -fmessage-length=0 -isystem /usr/local/miniconda/envs/test/include -march=core2 -mtune=haswell -mssse3 -ftree-vectorize -fPIC -fPIE -fstack-protector-strong -O2 -pipe -stdlib=libc++ -fvisibility-inlines-hidden -std=c++14 -fmessage-length=0 -isystem /usr/local/miniconda/envs/test/include -O3 -DNDEBUG -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk -bundle -Wl,-headerpad_max_install_names -Wl,-pie -Wl,-headerpad_max_install_names -Wl,-dead_strip_dylibs -Wl,-rpath,/usr/local/miniconda/envs/test/lib -L/usr/local/miniconda/envs/test/lib -o lib/yarp/yarp_ros2test.so src/devices/ros2test/CMakeFiles/yarp_ros2test.dir/yarp_plugin_ros2test.cpp.o src/devices/ros2test/CMakeFiles/yarp_ros2test.dir/Ros2Test.cpp.o  /usr/local/miniconda/envs/test/lib/libYARP_dev.3.5.0.dylib  /usr/local/miniconda/envs/test/lib/librclcpp.dylib  /usr/local/miniconda/envs/test/lib/libstd_msgs__rosidl_typesupport_c.dylib  /usr/local/miniconda/envs/test/lib/libYARP_math.3.5.0.dylib  /usr/local/miniconda/envs/test/lib/libYARP_sig.3.5.0.dylib  /usr/local/miniconda/envs/test/lib/libYARP_os.3.5.0.dylib  /usr/local/miniconda/envs/test/lib/libament_index_cpp.dylib  /usr/local/miniconda/envs/test/lib/liblibstatistics_collector.dylib  /usr/local/miniconda/envs/test/lib/librcl.dylib  /usr/local/miniconda/envs/test/lib/librcl_interfaces__rosidl_typesupport_introspection_c.dylib  /usr/local/miniconda/envs/test/lib/librcl_interfaces__rosidl_generator_c.dylib  /usr/local/miniconda/envs/test/lib/librcl_interfaces__rosidl_typesupport_c.dylib  /usr/local/miniconda/envs/test/lib/librcl_interfaces__rosidl_typesupport_introspection_cpp.dylib  /usr/local/miniconda/envs/test/lib/librcl_interfaces__rosidl_typesupport_cpp.dylib  /usr/local/miniconda/envs/test/lib/librmw_implementation.dylib  /usr/local/miniconda/envs/test/lib/librcl_logging_spdlog.dylib  /usr/local/miniconda/envs/test/lib/librcl_logging_interface.dylib  /usr/local/miniconda/envs/test/lib/librcl_yaml_param_parser.dylib  /usr/local/miniconda/envs/test/lib/librmw.dylib  /usr/local/miniconda/envs/test/lib/libyaml.dylib  /usr/local/miniconda/envs/test/lib/librosgraph_msgs__rosidl_typesupport_introspection_c.dylib  /usr/local/miniconda/envs/test/lib/librosgraph_msgs__rosidl_generator_c.dylib  /usr/local/miniconda/envs/test/lib/librosgraph_msgs__rosidl_typesupport_c.dylib  /usr/local/miniconda/envs/test/lib/librosgraph_msgs__rosidl_typesupport_introspection_cpp.dylib  /usr/local/miniconda/envs/test/lib/librosgraph_msgs__rosidl_typesupport_cpp.dylib  /usr/local/miniconda/envs/test/lib/libstatistics_msgs__rosidl_typesupport_introspection_c.dylib  /usr/local/miniconda/envs/test/lib/libstatistics_msgs__rosidl_generator_c.dylib  /usr/local/miniconda/envs/test/lib/libstatistics_msgs__rosidl_typesupport_c.dylib  /usr/local/miniconda/envs/test/lib/libstatistics_msgs__rosidl_typesupport_introspection_cpp.dylib  /usr/local/miniconda/envs/test/lib/libstatistics_msgs__rosidl_typesupport_cpp.dylib  /usr/local/miniconda/envs/test/lib/libtracetools.dylib  /usr/local/miniconda/envs/test/lib/libbuiltin_interfaces__rosidl_typesupport_introspection_c.dylib  /usr/local/miniconda/envs/test/lib/libbuiltin_interfaces__rosidl_generator_c.dylib  /usr/local/miniconda/envs/test/lib/libbuiltin_interfaces__rosidl_typesupport_c.dylib  /usr/local/miniconda/envs/test/lib/libbuiltin_interfaces__rosidl_typesupport_introspection_cpp.dylib  /usr/local/miniconda/envs/test/lib/librosidl_typesupport_introspection_cpp.dylib  /usr/local/miniconda/envs/test/lib/librosidl_typesupport_introspection_c.dylib  /usr/local/miniconda/envs/test/lib/libbuiltin_interfaces__rosidl_typesupport_cpp.dylib  /usr/local/miniconda/envs/test/lib/librosidl_typesupport_cpp.dylib  /usr/local/miniconda/envs/test/lib/librosidl_typesupport_c.dylib  /usr/local/miniconda/envs/test/lib/librcpputils.dylib  /usr/local/miniconda/envs/test/lib/librosidl_runtime_c.dylib  /usr/local/miniconda/envs/test/lib/librcutils.dylib && :
ld: warning: -pie being ignored. It is only used when linking a main executable
Undefined symbols for architecture x86_64:
  "rosidl_message_type_support_t const* rosidl_typesupport_cpp::get_message_type_support_handle<std_msgs::msg::String_<std::__1::allocator<void> > >()", referenced from:
      rclcpp::Publisher<std_msgs::msg::String_<std::__1::allocator<void> >, std::__1::allocator<void> >::Publisher(rclcpp::node_interfaces::NodeBaseInterface*, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, rclcpp::QoS const&, rclcpp::PublisherOptionsWithAllocator<std::__1::allocator<void> > const&) in Ros2Test.cpp.o
ld: symbol(s) not found for architecture x86_64
clang-11: error: linker command failed with exit code 1 (use -v to see invocation)

and 

 Generating Code...
     Creating library D:/a/yarp-ros2/yarp-ros2/build/lib/Release/yarp_ros2test.lib and object D:/a/yarp-ros2/yarp-ros2/build/lib/Release/yarp_ros2test.exp
Ros2Test.obj : error LNK2019: unresolved external symbol "struct rosidl_message_type_support_t const * __cdecl rosidl_typesupport_cpp::get_message_type_support_handle<struct std_msgs::msg::String_<class std::allocator<void> > >(void)" (??$get_message_type_support_handle@U?$String_@V?$allocator@X@std@@@msg@std_msgs@@@rosidl_typesupport_cpp@@YAPEBUrosidl_message_type_support_t@@XZ) referenced in function "public: __cdecl rclcpp::Publisher<struct std_msgs::msg::String_<class std::allocator<void> >,class std::allocator<void> >::Publisher<struct std_msgs::msg::String_<class std::allocator<void> >,class std::allocator<void> >(class rclcpp::node_interfaces::NodeBaseInterface *,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &,class rclcpp::QoS const &,struct rclcpp::PublisherOptionsWithAllocator<class std::allocator<void> > const &)" (??0?$Publisher@U?$String_@V?$allocator@X@std@@@msg@std_msgs@@V?$allocator@X@std@@@rclcpp@@QEAA@PEAVNodeBaseInterface@node_interfaces@1@AEBV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@AEBVQoS@1@AEBU?$PublisherOptionsWithAllocator@V?$allocator@X@std@@@1@@Z) [D:\a\yarp-ros2\yarp-ros2\build\src\devices\ros2test\yarp_ros2test.vcxproj]
D:\a\yarp-ros2\yarp-ros2\build\lib\yarp\Release\yarp_ros2test.dll : fatal error LNK1120: 1 unresolved externals [D:\a\yarp-ros2\yarp-ros2\build\src\devices\ros2test\yarp_ros2test.vcxproj]

I may be wrong, but it seems that indeed the library that provides the symbol struct rosidl_message_type_support_t const * __cdecl rosidl_typesupport_cpp::get_message_type_support_handle<struct std_msgs::msg::String_<class std::allocator<void> > >(void) is not linked, and everything works on Linux only because ELF is much more forgiving to missing symbols in linking phase.

traversaro commented 3 years ago

I may be wrong, but it seems that indeed the library that provides the symbol struct rosidl_message_type_support_t const * __cdecl rosidl_typesupport_cpp::get_message_type_support_handle<struct std_msgs::msg::String_<class std::allocator<void> > >(void) is not linked, and everything works on Linux only because ELF is much more forgiving to missing symbols in linking phase.

Indeed, there was a typo in the linked library that was fixed in https://github.com/robotology-playground/yarp-ros2/pull/11/commits/36f4e4d0a77839755b15c8d9a3a2663e6ce94bc8 . Now macOS builds work.

traversaro commented 3 years ago

Ok, now Windows fails for a classical reason: use of M_PI.

traversaro commented 3 years ago

CI is finally green, the main modifications are:

Let me know if you prefer separate PRs for the fixes, or if I can rebase the PR separating this three changes in three commits. @drdanz @ste93

drdanz commented 3 years ago

3 commits is fine! Let's merge this! Thanks @traversaro

drdanz commented 3 years ago

Oh it was actually 11 commits... Oh well 😄

traversaro commented 3 years ago

Yes, I actually also had to rebase due to https://github.com/robotology-playground/yarp-ros2/pull/11#issuecomment-905371735, but I guess that now it's done. :D

drdanz commented 3 years ago

Yeah, sorry about that 😅 Not a big deal anyway

randaz81 commented 2 years ago

workaround not useful anymore, removed.