Skip to content

Updated Docker Containers#2410

Open
N8BWert wants to merge 2 commits intoros2from
docker-update
Open

Updated Docker Containers#2410
N8BWert wants to merge 2 commits intoros2from
docker-update

Conversation

@N8BWert
Copy link
Copy Markdown
Contributor

@N8BWert N8BWert commented Sep 14, 2025

Description

This PR creates updated docker containers to make development a bit easier. It also introduces the usage of export MAKEFLAGS="-j 41" to the ubuntu-setup script so colcon is forced to only use a singular core.

Steps to Test

Linux

  1. Run ./docker/docker-build.sh
  2. Run ./docker/docker-run-linux.sh
  3. Enjoy the stack dockerized

Mac

  1. Download an X server (basically a unix server that allows windows to be shared across programs) such as XQuartz.
  2. Run ./docker/docker-build.sh
  3. Run ./docker/docker-run.mac.sh
  4. Enjoy the stack dockerized

Windows

  1. For windows, just use wsl
  2. If you really want to use the docker container get an X server like on mac
  3. Run ./docker/docker-build.sh
  4. Run ./docker/docker-run-windows.sh
  5. Enjoy the stack dockerized.

**Expected result:**You should be able to develop seamlessly within the docker container.

Review Checklist

  • Docstrings: All methods and classes should have the file appropriate docstrings which follow the guidelines in the "Contributing" page of our docs.
  • Remove extra print statements: Any print statements used for debugging should be removed
  • Tag reviewers: Tag some people for review and ping them on Slack

@sanatd33 sanatd33 requested a review from Squid5678 September 14, 2025 18:02
@N8BWert N8BWert requested a review from sanatd33 September 14, 2025 18:03
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.

1 participant