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

Install all Tier 1 RMWs in containers #58

Closed
SteveMacenski opened this issue Nov 12, 2023 · 2 comments · Fixed by #59
Closed

Install all Tier 1 RMWs in containers #58

SteveMacenski opened this issue Nov 12, 2023 · 2 comments · Fixed by #59
Labels
enhancement New feature or request

Comments

@SteveMacenski
Copy link

Description

I'm looking to test code in a matrix with all Tier 1 RMWs. Currently only the default is installed so I can't simply change env in actions to use the one I desire. That seems easier than needing to compile each other RMW (e.g. cyclone) alongside the code every workflow by including it in a repos file since its not a strict package.xml dependency.

Completion Criteria

Update the docker containers to include all Tier 1 supported RMWs

@SteveMacenski
Copy link
Author

Or -- analogously, have the default examples have the steps

    - name: Install Tier 1 DDS
      run: sudo apt install ros-iron-rmw-cyclonedds-cpp ... -y

In them so its immediately clear. Perhaps also explicitly set the env var too to make it clear to folks the workflow to get what they want out of the box without thinking.

@christophebedard
Copy link
Member

#59 installs the other Tier 1 DDS implementation in the non-base Ubuntu images. I moved your suggestion to the action-ros-ci repo: ros-tooling/action-ros-ci#838

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

Successfully merging a pull request may close this issue.

2 participants