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

Build failing on melodic #31

Closed
mkhansenbot opened this issue Jul 14, 2018 · 7 comments
Closed

Build failing on melodic #31

mkhansenbot opened this issue Jul 14, 2018 · 7 comments

Comments

@mkhansenbot
Copy link

Is this package supported on melodic? I'm trying to build and getting this error.

/home/mkhansen/ros_ws/src/depthimage_to_laserscan/src/DepthImageToLaserScanNodelet.cpp:60:24: error: expected constructor, destructor, or type conversion before ‘(’ token
PLUGINLIB_DECLARE_CLASS(depthimage_to_laserscan, DepthImageToLaserScanNodelet, depthimage_to_laserscan::DepthImageToLaserScanNodelet, nodelet::Nodelet);
^
Has anyone built it for melodic?

@mikaelarguedas
Copy link
Contributor

I think you need to use #27 for it to work on melodic and it hasn't been merged or released yet.

@mkhansenbot
Copy link
Author

@mikaelarguedas - thanks for the pointer. I pulled your patch and it built for me. Can we get that merged?

@mikaelarguedas
Copy link
Contributor

Can we get that merged?

@chadrockey very gentle 🛎️

@mkhansenbot
Copy link
Author

@dirk-thomas - can we get #27 merged so this can build on Melodic?

@chadrockey
Copy link
Member

Sorry all, I uncovered this and took care of it.
ros/rosdistro#20212

If anyone wants to provide more love to this package let me know and I'll promote you. Send me an email if possible, I've had too many Github emails over the years and can't keep track of things. Some might sitll be bouncing @willowgarage.com to this day... :\

@sidshete
Copy link

Replace

PLUGINLIB_DECLARE_CLASS(depthimage_to_laserscan, DepthImageToLaserScanNodelet, depthimage_to_laserscan::DepthImageToLaserScanNodelet, nodelet::Nodelet);

with

PLUGINLIB_EXPORT_CLASS(depthimage_to_laserscan::DepthImageToLaserScanNodelet, nodelet::Nodelet);

this should work!!

@sidshete
Copy link

Is this package supported on melodic? I'm trying to build and getting this error.

/home/mkhansen/ros_ws/src/depthimage_to_laserscan/src/DepthImageToLaserScanNodelet.cpp:60:24: error: expected constructor, destructor, or type conversion before ‘(’ token PLUGINLIB_DECLARE_CLASS(depthimage_to_laserscan, DepthImageToLaserScanNodelet, depthimage_to_laserscan::DepthImageToLaserScanNodelet, nodelet::Nodelet); ^ Has anyone built it for melodic?

Replace

PLUGINLIB_DECLARE_CLASS(depthimage_to_laserscan, DepthImageToLaserScanNodelet, depthimage_to_laserscan::DepthImageToLaserScanNodelet, nodelet::Nodelet);

with

PLUGINLIB_EXPORT_CLASS(depthimage_to_laserscan::DepthImageToLaserScanNodelet, nodelet::Nodelet);

this should work!!

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

No branches or pull requests

4 participants