Skip to content

Commit

Permalink
[N/A] make install script fully automatic (#384)
Browse files Browse the repository at this point in the history
## Change Overview

Resolves #381. Now no user input is needed during the portion of the install script that installs `bosdyn_msgs`. 

## Testing Done

- [x] uninstalled `bosdyn_msgs` packages locally and verified that when running the install script on main I was prompted for a `y`
- [x] running the install script on this branch does not prompt me for `y` and successfully installs `bosdyn_msgs` packages

I needed the `--nox11` flag as when piping in the `yes`, as by default this opens another terminal session that doesn't register anything before the pipe.
  • Loading branch information
khughes-bdai authored May 16, 2024
1 parent 6247e4e commit 707995a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion install_spot_ros2.sh
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ sudo pip3 install --force-reinstall -v "setuptools==59.6.0"
# Install bosdyn_msgs - automatic conversions of BD protobufs to ROS messages
wget -q -O /tmp/ros-humble-bosdyn_msgs_${MSG_VERSION}-jammy_${ARCH}.run https://github.com/bdaiinstitute/bosdyn_msgs/releases/download/${MSG_VERSION}/ros-humble-bosdyn_msgs_${MSG_VERSION}-jammy_${ARCH}.run
chmod +x /tmp/ros-humble-bosdyn_msgs_${MSG_VERSION}-jammy_${ARCH}.run
sudo /tmp/ros-humble-bosdyn_msgs_${MSG_VERSION}-jammy_${ARCH}.run
yes | sudo /tmp/ros-humble-bosdyn_msgs_${MSG_VERSION}-jammy_${ARCH}.run --nox11
rm /tmp/ros-humble-bosdyn_msgs_${MSG_VERSION}-jammy_${ARCH}.run

# Install spot-cpp-sdk
Expand Down

0 comments on commit 707995a

Please sign in to comment.