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

WIP: update to pybind11 2.6.0 #13

Merged
merged 4 commits into from
Nov 6, 2020
Merged

WIP: update to pybind11 2.6.0 #13

merged 4 commits into from
Nov 6, 2020

Conversation

henryiii
Copy link
Contributor

@henryiii henryiii commented Nov 5, 2020

This is a rough guess on what updating to pybind11 2.6.0 would look like. I have not used catkin (and don't really understand why normal, regular tooling can't be used...), but here's a suggestion for where to start.

@henryiii
Copy link
Contributor Author

henryiii commented Nov 6, 2020

This is only meant to be a staring point, and I may have removed too much. A few points: You can customize where the CMake and include files go (see example above), and you should make sure the relative positioning is correct so that CMake can correctly discover the relative locations. You should include all of the CMake files, picking a few files out and using them in isolation is not supported.

Please continue to work on this, I just wanted to provide a starting point.

@henryiii
Copy link
Contributor Author

henryiii commented Nov 6, 2020

For the included CMake file, I would ideally use the generated config file, but if that doesn't work, Iwould look at the Targets.CMake.in file and base your file on that.

@wxmerkt
Copy link
Owner

wxmerkt commented Nov 6, 2020

Hi @henryiii,
Thanks for the PR! I am working on this locally since the changes will break downstream builds and we need to consider different workspace scenarios (devel / install). The upstream version in the ROS binaries is working (and I've commented as such in the linked issue).

What's the current / new minimum required CMake version for pybind11 in v2.6.0? I investigated upgrading back on July 31st but the new minimum required CMake version of 3.8 conflicts with some of our system requirements (to support 16.04, 18.04 and 20.04 with stock CMake) - which is why I didn't push an update yet.

Thanks,
Wolfgang

@henryiii
Copy link
Contributor Author

henryiii commented Nov 6, 2020

3.4 on Linux, exactly so that we can support 16.04.

@wxmerkt
Copy link
Owner

wxmerkt commented Nov 6, 2020

I modified the PR such that it works on my machine - I think this should be good to merge. Thank you for the contribution!

@wxmerkt wxmerkt merged commit b749719 into wxmerkt:master Nov 6, 2020
@henryiii henryiii deleted the patch-1 branch November 6, 2020 02:38
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.

2 participants