-
Notifications
You must be signed in to change notification settings - Fork 302
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
Use ignition::gazebo:: in class instantiation #1967
Conversation
Fixes macOS build. Signed-off-by: Steve Peters <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-gazebo3 #1967 +/- ##
===============================================
+ Coverage 77.96% 78.04% +0.07%
===============================================
Files 255 255
Lines 15080 15080
===============================================
+ Hits 11757 11769 +12
+ Misses 3323 3311 -12
|
macOS build doesn't seem to like that. |
it's because I removed broken bottles in osrf/homebrew-simulation#2242, which maybe I didn't have to do as a separate step? At any rate, rebuilding the gz-common3 bottle in osrf/homebrew-simulation#2245 which should fix it |
homebrew build is now unstable, I believe because it ran on macos-monterey and there are some failing tests due to gazebosim/gz-physics#442 |
I believe the ABI checker is failing due to #1971 |
CI is 🟢 ! Hooray! |
🦟 Bug fix
Fixes macOS build.
Summary
The macOS build has been broken since #1646:
In other packages, I have avoided using the
gz::sim
namespace when implementing classes, so doing it here seems to fix the macOS build. The ABI build is broken too, but that needs a different fix.Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.