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 broken build by vendoring pkgs #170

Closed
wants to merge 6 commits into from
Closed

fix broken build by vendoring pkgs #170

wants to merge 6 commits into from

Conversation

koonpeng
Copy link

@koonpeng koonpeng commented Dec 1, 2022

Bug fix

Fixed bug

This is based on https://github.com/PranayShirodkar/rmf_demos/tree/feature/add_speed_limits however the same problems exist on the main branch. rmf_demos_fleet_adapter uses a number of python packages not available in ros humble, these packages are not declared in package.xml so it will not work without manually installing those packages.

There will be errors like:

ModuleNotFoundError: No module named 'fastapi'

Fix applied

Most of the packages are available in ubuntu repos but they are outdated, declaring them in package.xml will cause an incompatible version to be installed. The fix applied is to instead vendor these packages.

@luca-della-vedova
Copy link
Member

As of Ubuntu 24.04 the package mismatch issue of fastapi has been addressed, and with #211 they are specified as dependencies and installed from apt so this should no longer be needed.

@luca-della-vedova luca-della-vedova deleted the kp/vendor-pkgs branch May 30, 2024 14:07
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.

4 participants