Skip to content
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

move RTI Connext DDS installation doc under Install-Connext-Universit… #4774

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

Conversation

fujitatomoya
Copy link
Collaborator

…y-Eval.

closes #3573

@fujitatomoya
Copy link
Collaborator Author

@clalancette @ahcorde what do you think about this?

@@ -10,6 +10,7 @@ The default DDS vendor is eProsima's Fast DDS.

* :doc:`Working with Eclipse Cyclone DDS <DDS-Implementations/Working-with-Eclipse-CycloneDDS>` explains how to utilize Cyclone DDS.
* :doc:`Working with eProsima Fast DDS <DDS-Implementations/Working-with-eProsima-Fast-DDS>` explains how to utilize Fast DDS.
* :doc:`Working with RTI Connext DDS <DDS-Implementations/Install-Connext-University-Eval>` explains how to utilize and evaluate RTI Connext DDS.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keep this page for overview and introduction for each RMW implementation.

@@ -24,3 +28,174 @@ To install RTI Connext DDS **version 6.0.1** Evalution:
* Contact [email protected] for an evaluation license.
* Install RTI Connext 6.0.1 by running the installation program. When finished, it will run the RTI Launcher.
* Use the RTI Launcher to install the license file (rti_license.dat) if needed. The launcher may also be used to launch the diagnostic tools and services.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move RTI Connext DDS specific procedure in the RTI Connext page.

@fujitatomoya fujitatomoya self-assigned this Sep 24, 2024
Copy link

HTML artifacts: https://github.com/ros2/ros2_documentation/actions/runs/11016469907/artifacts/1972213664.

To view the resulting site:

  1. Click on the above link to download the artifacts archive
  2. Extract it
  3. Open html-artifacts-4774/index.html in your favorite browser

@fujitatomoya
Copy link
Collaborator Author

@ahcorde thanks for the review. let's hold this to merge for now, i would like @clalancette 's review comment on this.

@fujitatomoya
Copy link
Collaborator Author

@clalancette can you take a look when you have time?

@fujitatomoya
Copy link
Collaborator Author

@clalancette @audrow what do you think about this doc structure change for RTI connext DDS?

Comment on lines 8 to 9
A libraries-only version of RTI Connext DDS 6.0.1 may be installed per the :doc:`installation instructions <../../Installation>` for
Debian/Ubuntu Linux (amd64) platforms only, under a `non-commercial license <https://www.rti.com/ncl>`__.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the link to ../../Installation correct here? There is nothing about Connext on the Installation page. The Connext installation stuff is now on this page, so this reference to the generic installation instructions may not be needed.

@christophebedard
Copy link
Member

Minor comment aside, I personally think this makes a lot of sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RTI Connext DDS installation procedure needs to be reorganized
3 participants