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

update to use non deprecated pluginlib macro #630

Merged

Conversation

mikaelarguedas
Copy link
Contributor

These macros, deprecated for now 8 years, will be removed in the next ROS release (ROS Melodic)

I'm targeting the kinetic-devel branch as it is the default but I can target the lunar branch if it's preferred

@DLu DLu merged commit 5f41333 into ros-planning:kinetic-devel Nov 16, 2017
@mikaelarguedas mikaelarguedas deleted the update_pluginlib_macros branch November 16, 2017 18:36
DLu pushed a commit to DLu/navigation that referenced this pull request Nov 16, 2017
* update to use non deprecated pluginlib macro

* multiline version as well
DLu added a commit that referenced this pull request Nov 30, 2017
* Respect planner_frequency intended behavior (#622)

* Only do a getRobotPose when no start pose is given (#628)

Omit the unnecessary call to getRobotPose when the start pose was
already given, so that move_base can also generate a path in
situations where getRobotPose would fail.

This is actually to work around an issue of getRobotPose randomly
failing.

* Update gradient_path.cpp (#576)

* Update gradient_path.cpp

* Update navfn.cpp

* update to use non deprecated pluginlib macro (#630)

* update to use non deprecated pluginlib macro

* multiline version as well

* Print SDL error on IMG_Load failure in server_map (#631)
gerkey pushed a commit to codebot/navigation that referenced this pull request Jan 19, 2018
* update to use non deprecated pluginlib macro

* multiline version as well
johaq pushed a commit to CentralLabFacilities/navigation that referenced this pull request Mar 30, 2018
* update to use non deprecated pluginlib macro

* multiline version as well
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.

None yet

2 participants