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 fail on fedora 28 #2282

Closed
MalcolmMielle opened this issue Apr 17, 2018 · 1 comment
Closed

Build fail on fedora 28 #2282

MalcolmMielle opened this issue Apr 17, 2018 · 1 comment

Comments

@MalcolmMielle
Copy link

building pcl 1.8 on fedora 28 fails with:


In file included from /home/malcolm/external_sources/pcl/segmentation/include/pcl/segmentation/organized_multi_plane_segmentation.h:48,
                 from /home/malcolm/external_sources/pcl/segmentation/src/organized_multi_plane_segmentation.cpp:42:
/home/malcolm/external_sources/pcl/segmentation/include/pcl/segmentation/plane_coefficient_comparator.h: In member function ‘const std::vector<float>& pcl::PlaneCoefficientComparator<PointT, PointNT>::getPlaneCoeffD() const’:
/home/malcolm/external_sources/pcl/segmentation/include/pcl/segmentation/plane_coefficient_comparator.h:144:31: error: invalid initialization of reference of type ‘const std::vector<float>&’ from expression of type ‘const boost::shared_ptr<std::vector<float> >’
         return (plane_coeff_d_);
                               ^
make[2]: *** [segmentation/CMakeFiles/pcl_segmentation.dir/build.make:167: segmentation/CMakeFiles/pcl_segmentation.dir/src/organized_multi_plane_segmentation.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1102: segmentation/CMakeFiles/pcl_segmentation.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
  • Operating System and version: Fedora 28
  • Compiler: gcc8
  • PCL Version: 1.8
@taketwo taketwo added the needs: code review Specify why not closed/merged yet label Apr 17, 2018
@taketwo taketwo closed this as completed in 456d9b4 May 6, 2018
@taketwo taketwo removed the needs: code review Specify why not closed/merged yet label May 6, 2018
@taketwo
Copy link
Member

taketwo commented May 6, 2018

Fixed by #2299.

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

2 participants