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

Evalg 84 #711

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Evalg 84 #711

wants to merge 8 commits into from

Conversation

xFrenchy
Copy link
Contributor

Summary

Rewriting the EvalGuide Docker module to use Debian package for creating docker images with Connext installed

Details and comments

-Updated Dockerfile to install packages such as curl, run the basic install commands for Connext
-Updated CmakeList to refer to /app to find cmake files

Checks

  • [ x ] I have updated the documentation accordingly.
  • [ x ] I have read the CONTRIBUTING document.

tutorials/docker/CMakeLists.txt Outdated Show resolved Hide resolved
tutorials/docker/Dockerfile Outdated Show resolved Hide resolved
tutorials/docker/Dockerfile Outdated Show resolved Hide resolved
&& apt-get update \
&& apt-get install -y rti-connext-dds-7.3.0 \
&& apt-get clean \
&& rm -rf /var/lib/apt/lists

# For armv8 uncomment the following line
# ENV CONNEXTDDS_ARCH=armv8Linux4gcc7.3.0
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be nice to figure out the right CONNEXTDDS_ARCH based on the system's archicteure (for ex. with uname -m)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I haven't tested this on armv8, this comment was leftover from how the file was before. If everything else looks good, I'll test this part in the lab to see if setting CONNEXTDDS_ARCH is needed and if so, using uname -m or some other cpu command

tutorials/docker/Dockerfile Show resolved Hide resolved
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.

2 participants