-
Notifications
You must be signed in to change notification settings - Fork 7
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
Switch to Humble/Garden on 22.04 #22
Conversation
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Michael Carroll <michael@openrobotics.org>
|
will need to switch checkouts in the vcs yaml back to main after osrf/mbari_wec_gz#111 and osrf/mbari_wec_utils#38 are merged |
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.
Dockerfile built for me (once I replaced the buoy_all.yaml URL with the temporary file in this PR).
One comment for building from source. (The Dockerfile built without this because building from binaries sets up Python fine, but building from source needs a manual export I believe).
Will approve after the one change.
Signed-off-by: Michael Anderson <anderson@mbari.org>
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.
Since we are not using the default DDS in Humble, we should make it clear what our recommended configuration is.
Dockerfile addition? |
Signed-off-by: Michael Anderson <anderson@mbari.org>
…y_entrypoint into chapulina/humble_garden
updated |
Signed-off-by: Michael Anderson <anderson@mbari.org>
Quick review for github actions / ci