-
Notifications
You must be signed in to change notification settings - Fork 81
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
Update instructions for DDS Video #704
base: main
Are you sure you want to change the base?
Conversation
0e5ce70
to
c87c6ad
Compare
Signed-off-by: Victor Chang <[email protected]>
Signed-off-by: Victor Chang <[email protected]>
Signed-off-by: Victor Chang <[email protected]>
c87c6ad
to
6e4e9e1
Compare
Signed-off-by: Victor Chang <[email protected]>
Signed-off-by: Victor Chang <[email protected]>
applications/dds/dds_video/README.md
Outdated
# 2. Build the container | ||
./dev_container build --docker_file applications/dds/dds_video/Dockerfile | ||
# 3. Configure RTI Environment | ||
eval $(rtienv -l rti_license.dat) |
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.
The rtienv
tool seems to only be available when RTI Connext is install via apt
. This could still be ok, but we'd need to (a) document that the apt
installation method should be used, and (b) ensure that the apt install method can still be used on IGX / aarch64.
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.
Thank you @ibstewart. I have updated the instructions to use a .run
installer. I have also added a quick start section to use ./dev_container build_and_run
Signed-off-by: Victor Chang <[email protected]>
applications/dds
to pave the way for the upcoming DDS-based sample applications.