Skip to content

Update Dockerfile#10

Open
divan510 wants to merge 1 commit intoarshadlab:masterfrom
divan510:master
Open

Update Dockerfile#10
divan510 wants to merge 1 commit intoarshadlab:masterfrom
divan510:master

Conversation

@divan510
Copy link
Contributor

The line 41:

Install dependencies using rosdep

RUN rosdep install --from-paths src --ignore-src -r -y

did not work, because of some inconsistencies due to ros2 version. The dockerfile would not build the image.
Had to add lines 17 to 21 for it to work.

For me now works, please check it.

P.S. This should resolve the rosdep install --from-paths src --ignore-src -r -y issue that is open.

The line 41:
# Install dependencies using rosdep
RUN rosdep install --from-paths src --ignore-src -r -y

did not work, because of some inconsistencies due to ros2 version.
The dockerfile would not build the image.
Had to add lines 17 to 21 for it to work.
@arshadlab
Copy link
Owner

I don't see any issue when building docker image on my machine. tried with latest code too.

@divan510
Copy link
Contributor Author

divan510 commented Sep 26, 2025

I don't know why, but to me, even now if I try to do it the way that you suggested, have the error:
ERROR: failed to build: failed to solve: process "/bin/sh -c rosdep install --from-paths src --ignore-src -r -y" did not complete successfully

Only if I do as I suggested in the Dockerfile, it works.

Edit:
I talked to other people that tried to use this repo, they had the same problem, so I suggest that you take into account my proposition.

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