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

Fix remaining issues in applications and demos from the updated spaceros base image #162

Closed
wants to merge 3 commits into from

Conversation

eholum
Copy link
Contributor

@eholum eholum commented Aug 11, 2024

Resolves #161.

Sorry for not having this queued up alongside https://github.com/space-ros/space-ros/pull/170/files. There are a few issues to resolve after blowing the base build up that I am working through here. Including fixing references to nonexistent directories and updating environment variables to the standards.

Copy link

@asimonov asimonov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@eholum
Copy link
Contributor Author

eholum commented Aug 12, 2024

@asimonov CI here is failing because it's building using the released versions of the space ros base image, not main. Will determine a path forward in slack, but until we have one I'll leave this in draft. If anyone else runs into issues like #160, this would resolve them.

@Bckempa
Copy link
Contributor

Bckempa commented Aug 22, 2024

Is space-ros/space-ros#193 the only blocker here or are we waiting on something else?

@eholum
Copy link
Contributor Author

eholum commented Aug 22, 2024

Is space-ros/space-ros#193 the only blocker here or are we waiting on something else?

It is, but once that goes in I'll also need to parameterize the base image name for the moveit and nav2 apps. Sorry this isn't moving as quickly as I'd like, been rather distracted lately.

@Bckempa
Copy link
Contributor

Bckempa commented Sep 9, 2024

This will also close #180 and has an open PR with a subset of the fixes in #177, FYI @dsobek.

@eholum
Copy link
Contributor Author

eholum commented Sep 19, 2024

Closing in favor of #189 for access to org packages.

@eholum eholum closed this Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Adjust READMEs and Apply Fixes for Demos Base Images
3 participants