usdot-fhwa-stol / carma-msgs

ROS Message definitions used by the CARMA Platform
6 stars 9 forks source link

Fix target directory for cloned ros1_bridge repository to ensure it is built in final image #234

Closed JonSmet closed 6 months ago

JonSmet commented 6 months ago

PR Details

Description

This PR updates checkout.bash to change the target directory in which the ros1_bridge repo is cloned into when the BRANCH environment variable is set to develop. Prior to this change, when BRANCH was set to develop, the ros1_bridge repo was cloned into /home/carma/src, but the install.sh script only builds ROS 2 packages located at ~/.base-image/ros2_msgs_ws

Related GitHub Issue

Related Jira Key

Motivation and Context

How Has This Been Tested?

CI (docker/docker action)

Types of changes

Checklist:

MishkaMN commented 6 months ago

This build is finishing fine, it's just that the github action is running out of space. Tested on this PR https://github.com/usdot-fhwa-stol/carma-msgs/pull/235

JonSmet commented 6 months ago

Closing this PR since the change is being included in #237