-
Notifications
You must be signed in to change notification settings - Fork 275
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
[WIP] Updating for nvidia-docker2 #40
Conversation
Dockerfile
Outdated
--from-paths . \ | ||
--ignore-src \ | ||
--rosdistro ${ROS_DISTRO} \ | ||
--skip-keys=prius_msgs \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure why --ignore-src
doesn't resolve this, but rosdep complaints without these exceptions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it in your workspace? It's only finding one package in the build.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would also explain the compile error too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But the prius_msgs
package was copied into the source workspace directory in line 17: https://github.com/ruffsl/car_demo/blob/4449922ec4a23218d08a7626fe866d1cf3fcadff/Dockerfile#L17
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, this was an issue with how I was copying in the source. now fixed.
For the build error do you have the patch from #39 catkin tools vs catkin_make could have different execution order causing race conditions. |
Ok, I got it to compile a further on 18.04 by roping in some missing packages from melodic just yet, but cmake is still not finding the
<\details> |
replaced by #47 |
This won't build yet for two reasons:
libglvnd0
is not yet released into trusyI'd like to bump this up to melodic to get
libglvnd0
, butfake-localization
has not yet been released.Here is the build issue, I'm not sure why
catkin_make
works here bycatkin_tools
doesn't.Build errors
ping @tfoote if you see a glaring error I made.