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

Fix compilation with Boost 1.73 #2758

Merged
merged 1 commit into from
Jul 27, 2020
Merged

Conversation

traversaro
Copy link
Collaborator

@traversaro traversaro commented Jun 15, 2020

The ModelTreeWidget.cc file was using the Boost Bind placeholders, but it was not including boost/bind.hpp . Before Boost 1.73, this was working probably due to transitive includes.

Slightly related issue: #2757 .

This change is required to build Gazebo11 with the latest version of vcpkg.

The ModelTreeWidget.cc file was using the Boost Bind placeholders, but it was not 
including boost/bind.hpp . Before Boost 1.73, this was working probably due to transitive
includes.

Slightly related issue: gazebosim#2757 . 

Signed-off-by: Silvio Traversaro <[email protected]>
@scpeters
Copy link
Member

we've often found these problems when homebrew-core updates the boost version, but boost 1.73.0 hasn't merged there yet: Homebrew/homebrew-core#53874

thanks for the fix!

@traversaro
Copy link
Collaborator Author

we've often found these problems when homebrew-core updates the boost version, but boost 1.73.0 hasn't merged there yet: Homebrew/homebrew-core#53874

thanks for the fix!

I found this completely by chance by updating a dependency built script for a project (that typically uses a fixed vcpkg version for reproducibility), but indeed it probably make sense to setup a weekly CI job that compile Gazebo and all its dependencies using the latest vcpkg, to early catch these kind of regressions, I willprobably set it up. The nice aspects is that for repos that have official ports on vcpkg we get this report even before the dependency is updated, see for example gazebosim/gz-msgs#56 .

@chapulina chapulina merged commit eba1c6f into gazebosim:gazebo11 Jul 27, 2020
@traversaro traversaro deleted the patch-2 branch September 9, 2020 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Gazebo 11
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants