Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use allocator via init_options argument. #2129

Merged
merged 1 commit into from
Mar 17, 2023

Conversation

fujitatomoya
Copy link
Collaborator

part of ros2/rcl#1036, but independent bug fix.

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Collaborator Author

@ros-pull-request-builder retest this please

@fujitatomoya
Copy link
Collaborator Author

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, the allocator is being used before we initialize init_options_. Switching to the passed-in init_options fixes it.

@clalancette
Copy link
Contributor

That said, I suggest we run CI with --packages-above rclcpp, because this touches the init code. Once that is green, I'm happy to merge this in.

@fujitatomoya fujitatomoya merged commit 1a796b5 into rolling Mar 17, 2023
@delete-merged-branch delete-merged-branch bot deleted the fujitatomoya/bugfix-rcl-issues-1036 branch March 17, 2023 19:35
@fujitatomoya
Copy link
Collaborator Author

@Mergifyio backport humble

@mergify
Copy link

mergify bot commented Mar 17, 2023

backport humble

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Mar 17, 2023
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 1a796b5)
@fujitatomoya
Copy link
Collaborator Author

@clalancette sorry i missed your comment, browser cache or something did not show it. i will start the CI as you suggested.

@fujitatomoya
Copy link
Collaborator Author

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@fujitatomoya
Copy link
Collaborator Author

all test passed but it show failing signs, i am not sure what went wrong.

@fujitatomoya
Copy link
Collaborator Author

I think build / test option was not correct.

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@fujitatomoya
Copy link
Collaborator Author

unstable failure is not related to this PR.

fujitatomoya added a commit that referenced this pull request Mar 20, 2023
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 1a796b5)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Barry-Xu-2018 pushed a commit to Barry-Xu-2018/rclcpp that referenced this pull request Jan 12, 2024
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants