Skip to content

Conversation

chen-harrison
Copy link
Contributor

@chen-harrison chen-harrison commented Aug 30, 2025

Summary

  • Requested by @azeey from docs: install_ubuntu_src gazebo_test_cases#2257
  • Include sudo apt update before install required packages
  • Add -y to 'sudo apt install' command for consistency with the rest of the documentation
  • More informative warning section about RAM usage above colcon build commands

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

- Include 'sudo apt update' before install required packages
- Add -y to 'sudo apt install' command for consistency with the rest
  of the documentation
- More informative warning section about RAM usage above colcon build
  commands

Signed-off-by: Harrison Chen <hchen.robotics@gmail.com>
@chen-harrison
Copy link
Contributor Author

I think these changes are also relevant for the equivalent page on previous versions of Gazebo, not just Jetty, so let me know if you'd prefer to have these changes applied to the equivalent pages for Ionic, Harmonic, etc.

@azeey
Copy link
Contributor

azeey commented Aug 30, 2025

Thanks for the PR! And yes, please update the ionic and harmonic folders as well.

- Add changes to 'Source Installation on Ubuntu' to Harmonic and Ionic
  versions of the documentation

Signed-off-by: Harrison Chen hchen.robotics@gmail.com
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Sep 2, 2025
@azeey azeey enabled auto-merge (squash) September 2, 2025 17:00
@azeey azeey merged commit 0f392e9 into gazebosim:master Sep 2, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants