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

Missing steps in Dockerfile for installing ros-dependencies #632

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

Conversation

nahsimai
Copy link

@nahsimai nahsimai commented Aug 9, 2022

  • Fixes missing steps for installing ros-dependencies.

If there are some corrections/improvements to be made, I will be happy to help further.


This change is Reviewable

@BastOOn58
Copy link

Since June 20th, 2023 the ROS Foxy Fitzroy distribution as reached its end-of-life (see here). As the ROS bridge works with this distribution, you must add the following argument to the rosdep update command to avoid any problem.

rosdep update --rosdistro=$ROS_DISTRO

@nahsimai
Copy link
Author

Since June 20th, 2023 the ROS Foxy Fitzroy distribution as reached its end-of-life (see here). As the ROS bridge works with this distribution, you must add the following argument to the rosdep update command to avoid any problem.

rosdep update --rosdistro=$ROS_DISTRO

Thanks for your comment @BastOOn58. I have made the relevant changes to the Dockerfile as suggested by you.

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