Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

fix: boost optional build error on rolling environment #241

Merged
merged 3 commits into from
Jun 14, 2024

Conversation

f0reachARR
Copy link
Contributor

@f0reachARR f0reachARR commented Mar 19, 2024

Description

As ROS 2 jazzy will be released soon, this PR will fix build error found in rolling build environment.

  • Add boost/optional/optional_io.hpp to some source files. This header file is required by newer version of Boost.

Tests performed

colcon test on humble distribution does not show new error.

Effects on system behavior

Not applicable.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

f0reachARR and others added 2 commits March 19, 2024 18:42
Signed-off-by: f0reachARR <f0reach@f0reach.me>
Copy link

stale bot commented May 18, 2024

This pull request has been automatically marked as stale because it has not had recent activity.

@stale stale bot added the status:stale Inactive or outdated issues. (auto-assigned) label May 18, 2024
@stale stale bot removed the status:stale Inactive or outdated issues. (auto-assigned) label May 31, 2024
@f0reachARR f0reachARR marked this pull request as ready for review May 31, 2024 03:24
@xmfcx xmfcx merged commit a7e776a into autowarefoundation:main Jun 14, 2024
25 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants