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

Revert "add line feed for RCUTILS_SAFE_FWRITE_TO_STDERR (#608)" #612

Conversation

fujitatomoya
Copy link
Collaborator

This reverts commit 7202ceb.

Signed-off-by: Tomoya Fujita Tomoya.Fujita@sony.com

This reverts commit 7202ceb.

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

@Blast545 can you check this revert just in case?

Copy link

@Blast545 Blast545 left a comment

Choose a reason for hiding this comment

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

Revert looks good to me

@Blast545
Copy link

@fujitatomoya FYI This test regression is affecting only one rmw, and it's tier 2 for ROS2 Humble. A revert is not mandatory for this one if you are going to be taking a look into it.

@fujitatomoya fujitatomoya merged commit 1017f15 into ros2:master Jun 17, 2022
@fujitatomoya
Copy link
Collaborator Author

@Blast545 thanks for checking.

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

2 participants