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 debian builds (closes #769) #770

Merged
merged 2 commits into from
Jul 29, 2018
Merged

Fix debian builds (closes #769) #770

merged 2 commits into from
Jul 29, 2018

Conversation

mikeferguson
Copy link
Contributor

No description provided.

previously, amcl depended on TF, which depended on
sensor_msgs.
@mikeferguson
Copy link
Contributor Author

First commit definitely belongs. Second is a bit of a wild guess since I can't figure out why the costmap_2d package isn't building. I compared with robot_calibration (something I released a few weeks ago and is building, and has similar depends), and it seems either it is the addition of Boost, or the removal of TF that is causing the issue....

@mikeferguson
Copy link
Contributor Author

Forget that first comment -- robot_calibration has same Boost line, so my second commit was definitely garbage...

I'm taking a closer look at what TF depends on...

@mikeferguson
Copy link
Contributor Author

Ok -- figured it out -- had to scroll back up further -- we have a find_package on tf2_geometry_msgs but it is not listed in package.xml

@mikeferguson mikeferguson merged commit d38644c into melodic-devel Jul 29, 2018
@mikeferguson mikeferguson deleted the fix_debians branch July 29, 2018 00:17
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.

1 participant