-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ros-{DISTRO}-rmw-connextdds should be installed. #3505
Conversation
Signed-off-by: Tomoya.Fujita <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@clalancette do we backport all here? |
@@ -43,7 +43,7 @@ You will need to accept a license from RTI. | |||
|
|||
.. code-block:: bash | |||
|
|||
sudo apt update && sudo apt install -q -y rti-connext-dds-6.0.1 | |||
sudo apt update && sudo apt install -q -y rti-connext-dds-6.0.1 ros-{DISTRO}-rmw-connextdds |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure that we should do this here. The text below seems to assume that we will be building the RMW from source, not installing it as a debian package.
I think really that this whole page needs to be revamped, since it is pretty confusing right now. We don't necessarily need to do that in here, but we should probably file a bug about it to follow up with later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good to me, will post the issue with description as follow-up.
follow-up issue #3573 created, i will go ahead to close this PR. |
part of osrf/ros2_test_cases#713