Repository navigation
[README.md] Suggest the curl | bash installer - #1
Merged
Merged
Conversation
Promote jazzy/install.sh to the recommended Ubuntu install path, keeping the manual apt instructions below it. Co-Authored-By: OpenHands <openhands@all-hands.dev>
Member
Author
Verification notes
This comment was generated by an AI agent (OpenHands) on behalf of Murilo Marinho. |
mmmarinho
had a problem deploying
to
github-pages
October 7, 2026 12:17 — with
GitHub Actions
Failure
This branch had an error being deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds a One-line installer subsection at the top of Ubuntu Installation, pointing at
jazzy/install.shviaraw.githubusercontent.com, and keeps the existing manual apt-get instructions below it as the alternative.curl -fsSL https://raw.githubusercontent.com/SmartArmStack/smart_arm_stack_ROS2/lyrical/jazzy/install.sh | bashWhy
The manual block is 6 lines of key/source/install steps that are easy to get wrong, and it duplicates what
install.shalready does idempotently. smart_arm_stack_ROS2 PR #12 makes that script safe to pipe intobashfrom a normal interactive shell: it validatessudocredentials up front (so the password prompt works when stdin is a pipe), detects the Ubuntu codename and architecture instead of hardcodingnoble, installs the OSRF Gazebo source its own packages need, and never runsapt-get upgrade/autoremove.Choices worth confirming
lyricalref plus thejazzy/subdirectory.lyricalis the repository default branch, so the link is stable and does not depend on branch naming. Note thatjazzy/install.sh(on thejazzybranch) is the old 17-line version and still fails: it adds no OSRF Gazebo source, soros-jazzy-sas-robot-driver-gazebodies on unmetlibgz-math7-dev/libsdformat14-dev. I verified both commands inghcr.io/marinholab/ros:jazzy— thejazzy-ref one fails, thelyrical/jazzy/one installs cleanly.ROS 2plusDQ Roboticsremain pre-requisites, since the script does not add theROS 2apt source.-nfor a dry run and that re-running is safe.Not done
No
CHANGELOG.mdentry — that file holds personal project-history notes rather than release notes. Say the word if you want one.This PR was created by an AI agent (OpenHands) on behalf of Murilo Marinho.