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

support melodic compile #249

Closed
wants to merge 24 commits into from

Conversation

Naoki-Hiraoka
Copy link
Contributor

@Naoki-Hiraoka Naoki-Hiraoka commented Apr 10, 2020

This PR includes #248.

gazebo::math is completely deprecated in melodic.

hydro, indigo and kinetic supports gazebo::math.
kinetic and melodic supports ignition::math
http://gazebosim.org/tutorials?tut=install_dependencies_from_source
http://gazebosim.org/tutorials/?tut=ros_wrapper_versions
https://bitbucket.org/osrf/gazebo/src/default/Migration.md?fileviewer=file-view-default

Since API of ignition::math is different from gazebo::math, we cannot use clever methods like 11de69e, so we have to write "#ifdef #endif everywhere gazebo::math is used.

@Naoki-Hiraoka Naoki-Hiraoka changed the title [WIP]support melodic compile [WIP][on #248]support melodic compile Apr 12, 2020
@Naoki-Hiraoka Naoki-Hiraoka changed the title [WIP][on #248]support melodic compile [on #248]support melodic compile Apr 13, 2020
@Naoki-Hiraoka Naoki-Hiraoka changed the title [on #248]support melodic compile support melodic compile Apr 13, 2020
@Naoki-Hiraoka
Copy link
Contributor Author

This PR is included in #250

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