build: add ros_environment dependency

[esteve]

PR Type

• Bug Fix

Related Links

https://github.com/autowarefoundation/autoware-deb-packages/actions/runs/5938935074/job/16104423980?pr=21

Description

When building in an isolated environment the ROS_DISTRO environment variable is only available if ros_environment is declared as a dependency in package.xml (see https://github.com/ros/ros_environment/blob/rolling/)

Review Procedure

Remarks

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

• Assign PR to reviewer

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

• Commits are properly organized and messages are according to the guideline
• (Optional) Unit tests have been written for new behavior
• PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

• All open points are addressed and tracked via issues or tickets

CI Checks

• Build and test for PR: Required to pass before the merge.

[esteve]

@amc-nu I don't have permissions to assign a reviewer for this PR, but could you have a look at these changes? The Debian packages builder is failing because ROS_DISTRO is not available at build time unless ros_environemnt is a dependency (see https://github.com/autowarefoundation/autoware-deb-packages/actions/runs/5938935074/job/16104423980?pr=21). Thanks.

[esteve]

@amc-nu pinging again, when I submitted the previous comment I tagged the wrong user.

[esteve]

@amc-nu thanks 🙂

[amc-nu]

Thanks @esteve
Looking forward to apt install ros-humble-autoware.