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

gazebo11 homebrew CI builds failing with cmake 3.22.0 #573

Closed
scpeters opened this issue Nov 24, 2021 · 2 comments
Closed

gazebo11 homebrew CI builds failing with cmake 3.22.0 #573

scpeters opened this issue Nov 24, 2021 · 2 comments

Comments

@scpeters
Copy link
Contributor

Since the latest version of tbb in homebrew-core has removed APIs that gazebo11 uses (gazebosim/gazebo-classic#2867), we have added an older version of tbb@2020_u3 to our homebrew-simulation tap (osrf/homebrew-simulation#1483) along with special handling for environment variables in CI scripts since this formula is not installed to /usr/local (#465 and #466). The gazebo-ci-gazebo11-homebrew-amd64 was passing as recently as November 16:

The job is now failing with error messages indicating that it is including the header files from the main tbb formula installed to /usr/local.

In file included from /Users/jenkins/workspace/gazebo-ci-gazebo11-homebrew-amd64/gazebo/gazebo/transport/ConnectionManager.cc:22:
In file included from /Users/jenkins/workspace/gazebo-ci-gazebo11-homebrew-amd64/gazebo/gazebo/transport/TopicManager.hh:35:
In file included from /Users/jenkins/workspace/gazebo-ci-gazebo11-homebrew-amd64/gazebo/gazebo/transport/SubscriptionTransport.hh:24:
/Users/jenkins/workspace/gazebo-ci-gazebo11-homebrew-amd64/gazebo/gazebo/transport/Connection.hh:57:65: error: expected class name
    class GZ_TRANSPORT_VISIBLE ConnectionReadTask : public tbb::task
                                                                ^
/Users/jenkins/workspace/gazebo-ci-gazebo11-homebrew-amd64/gazebo/gazebo/transport/Connection.hh:73:20: error: no type named 'task' in namespace 'tbb'
      public: tbb::task *execute()
              ~~~~~^

I believe the failure has occurred since cmake 3.22.0 was merged in Homebrew/homebrew-core#89638 based on tests on my local machine with cmake 3.21.0 and 3.22.0.

It's hard to avoid installing the main tbb formula since ogre1.9 uses the one in homebrew-core. Some possible solutions:

@scpeters
Copy link
Contributor Author

As a workaround, I've added a [email protected] formula to our tap in osrf/homebrew-simulation#1696 that we can use for gazebo CI

scpeters added a commit that referenced this issue Nov 30, 2021
gazebo11 needs an older version of cmake on homebrew,
but it is keg-only so prepend its bin folder to the PATH.
Workaround for #573.

Signed-off-by: Steve Peters <[email protected]>
scpeters added a commit that referenced this issue Nov 30, 2021
gazebo11 needs an older version of cmake on homebrew,
but it is keg-only so prepend its bin folder to the PATH.
Workaround for #573.

Signed-off-by: Steve Peters <[email protected]>
@scpeters
Copy link
Contributor Author

using cmake 3.21.4 fixes it for now

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

1 participant