Skip to content
This repository has been archived by the owner on Oct 30, 2024. It is now read-only.

add to support Cyclone DDS #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

atinfinity
Copy link
Contributor

I add to support Cyclone DDS.

Test

I checked on the following environment.

  • macOS Sonoma 14.0
  • M1 MacBook Air
export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp
ros2 topic pub --rate 30 /cmd_vel geometry_msgs/msg/Twist "{linear: {x: 2.0, y: 0.0, z: 0.0}, angular: {x: 0.0, y: 0.0, z: 1.8}}"
export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp
ros2 topic echo /cmd_vel --no-arr

@TakanoTaiga
Copy link
Owner

Thank you for the Pull Request. Upon checking, I encountered the following error and couldn't build in my environment. Could there be any mistakes in the procedure? I will also investigate the cause on my end.

/Users/taiga/Desktop/pr_ws/ros2_m1_native/src/eclipse-cyclonedds/cyclonedds/src/security/api/../../core/ddsi/include/dds/ddsi/ddsi_security_omg.h:410:63: warning: empty paragraph passed to '@retval' command [-Wdocumentation]
 * @retval NN_ENTITYID_SEDP_BUILTIN_SUBSCRIPTIONS_SECURE_WRITER
   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
/Users/taiga/Desktop/pr_ws/ros2_m1_native/src/eclipse-cyclonedds/cyclonedds/src/security/api/../../core/ddsi/include/dds/ddsi/ddsi_security_omg.h:411:56: warning: empty paragraph passed to '@retval' command [-Wdocumentation]
 * @retval NN_ENTITYID_SEDP_BUILTIN_SUBSCRIPTIONS_WRITER
   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
/Users/taiga/Desktop/pr_ws/ros2_m1_native/src/eclipse-cyclonedds/cyclonedds/src/security/api/../../core/ddsi/include/dds/ddsi/ddsi_security_omg.h:425:48: warning: empty paragraph passed to '@retval' command [-Wdocumentation]
 * @retval NN_ENTITYID_SEDP_BUILTIN_TOPIC_WRITER
   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
5 warnings generated.
dyld[18109]: Library not loaded: @rpath/libiceoryx_posh.dylib
  Referenced from: <9E10728F-816E-38D0-83CA-4B804AE5AAE7> /Users/taiga/Desktop/pr_ws/ros2_m1_native/install/iceoryx_binding_c/lib/libiceoryx_binding_c.dylib
  Reason: tried: '/Users/taiga/Desktop/pr_ws/ros2_m1_native/install/iceoryx_binding_c/lib/libiceoryx_posh.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/taiga/Desktop/pr_ws/ros2_m1_native/install/iceoryx_binding_c/lib/libiceoryx_posh.dylib' (no such file), '/Users/taiga/Desktop/pr_ws/ros2_m1_native/build/cyclonedds/lib/libiceoryx_posh.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/taiga/Desktop/pr_ws/ros2_m1_native/build/cyclonedds/lib/libiceoryx_posh.dylib' (no such file), '/Users/taiga/Desktop/pr_ws/ros2_m1_native/build/cyclonedds/lib/libiceoryx_posh.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/taiga/Desktop/pr_ws/ros2_m1_native/build/cyclonedds/lib/libiceoryx_posh.dylib' (no such file)
/bin/sh: line 1: 18109 Abort trap: 6           ../../../bin/idlc -Wno-implicit-extensibility -o/Users/taiga/Desktop/pr_ws/ros2_m1_native/build/cyclonedds/src/tools/ddsperf /Users/taiga/Desktop/pr_ws/ros2_m1_native/src/eclipse-cyclonedds/cyclonedds/src/tools/ddsperf/ddsperf_types.idl

@atinfinity
Copy link
Contributor Author

@TakanoTaiga

/bin/sh: line 1: 18109

Did you use ros2_m1_native/setup.sh to build?
I forgot to maintain this script.

@TakanoTaiga
Copy link
Owner

@atinfinity
I did not use ros2_m1_native/setup.sh. This script is intended for purposes like CI/CD and does not require maintenance. The issue with "/bin/sh: line 1: 18109" is likely due to not using the macOS standard terminal, I believe.

@atinfinity
Copy link
Contributor Author

@TakanoTaiga I uninstalled Homebrew. And, I built according to install_guide.md
But, I could not reproduce this error.
My environment is as follows.

  • macOS Sonoma 14.0
  • Homebrew 4.2.1
  • Xcode 15.1
  • Shell: zsh
  • Terminal: macOS standard terminal
    • I tried to use iTerm. But, I could not reproduce this error.

Could you provide build script to reproduce this error?

hanchangyo referenced this pull request in hanchangyo/ros2_humble_m1 Aug 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants