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

Compatibility with Boost 1.82 #412

Closed
SMillerDev opened this issue May 8, 2023 · 3 comments · Fixed by #418
Closed

Compatibility with Boost 1.82 #412

SMillerDev opened this issue May 8, 2023 · 3 comments · Fixed by #418

Comments

@SMillerDev
Copy link

SMillerDev commented May 8, 2023

When trying to build hpp-fcl for Boost 1.82 in Homebrew we got the following error:

  In file included from /tmp/hpp-fcl-20230507-45114-cgp1d1/hpp-fcl-2.3.2/python/broadphase/broadphase.cc:60:
  /tmp/hpp-fcl-20230507-45114-cgp1d1/hpp-fcl-2.3.2/python/broadphase/broadphase_collision_manager.hh:217:37: error: no member named 'typeindex' in namespace 'boost'
      std::string class_name = boost::typeindex::type_id<Derived>().pretty_name();
                               ~~~~~~~^
  /tmp/hpp-fcl-20230507-45114-cgp1d1/hpp-fcl-2.3.2/python/broadphase/broadphase_collision_manager.hh:217:56: error: 'Derived' does not refer to a value
      std::string class_name = boost::typeindex::type_id<Derived>().pretty_name();
                                                         ^
  /tmp/hpp-fcl-20230507-45114-cgp1d1/hpp-fcl-2.3.2/python/broadphase/broadphase_collision_manager.hh:215:22: note: declared here
    template <typename Derived>
                       ^
  /tmp/hpp-fcl-20230507-45114-cgp1d1/hpp-fcl-2.3.2/python/broadphase/broadphase_collision_manager.hh:217:65: error: expected expression
      std::string class_name = boost::typeindex::type_id<Derived>().pretty_name();
                                                                  ^

Which seems to be caused by https://github.com/humanoid-path-planner/hpp-fcl/blob/devel/python/broadphase/broadphase_collision_manager.hh#L215-L225

@jcarpent
Copy link
Contributor

jcarpent commented May 8, 2023

Dear @SMillerDev,

Thanks for raising this issue.
It has been solved in #401.
I will make a new release today and the homebrew formula will be updated then.

@SMillerDev Is it fine on your side?

@SMillerDev
Copy link
Author

Ah, missed that when searching for it in issues. Yeah, a new release sounds good to me!

@jcarpent
Copy link
Contributor

jcarpent commented May 8, 2023

The release is out.

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 a pull request may close this issue.

2 participants