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

Update instructions for building Gazebo from source on macOS #486

Merged
merged 4 commits into from
Aug 15, 2024

Conversation

azeey
Copy link
Contributor

@azeey azeey commented Aug 14, 2024

Credit goes to @srmainwaring who posted the necessary updates in a comment during the last Gazebo tutorial party.

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.

@azeey azeey requested a review from mabelzhang as a code owner August 14, 2024 20:50
@github-actions github-actions bot added the 🎵 harmonic Gazebo Harmonic label Aug 14, 2024
@azeey azeey requested a review from scpeters August 14, 2024 20:52
Copy link
Member

@scpeters scpeters left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes this is correct and should be propagated to versions from fortress to ionic (not citadel)

harmonic/install_osx_src.md Outdated Show resolved Hide resolved
@github-actions github-actions bot added 🌱 garden Gazebo Garden 🏯 fortress Ignition Fortress labels Aug 15, 2024
@azeey
Copy link
Contributor Author

azeey commented Aug 15, 2024

I've progated the changes to other releases. We also need to do the same for #414

@azeey azeey merged commit ac943c4 into gazebosim:master Aug 15, 2024
5 checks passed
@azeey azeey deleted the update_macos_instructions branch August 15, 2024 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏯 fortress Ignition Fortress 🌱 garden Gazebo Garden 🎵 harmonic Gazebo Harmonic
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants