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

Backport "Force a write of all user-space buffered data for stdout" to kinetic? #24

Closed
howardcochran opened this issue Jan 8, 2019 · 1 comment
Labels
question Further information is requested

Comments

@howardcochran
Copy link

The commit 71905d7, "Force a write of all user-space buffered data for stdout in Formatter::print" added the behavior of flushing stdout for each ROS_LOG when logging to stdout and when the environment variable ROSCONSOLE_STDOUT_LINE_BUFFERED is set to "1". This behavior is critical for getting timely ROS messages whenever stdout is a pipe (e.g. when it is connected to the systemd journal, or as the author of the commit mentioned, when running in Android).

I would like access to this feature in kinetic. Since it causes no behavior change unless the referenced environment variable is set, it should cause no compatibility issue. The patch applies cleanly (after accounting for the rename associated with moving rosconsole out of ros_comm to its own repo).

What is the procedure for getting this change into kinetic in the official release?

@dirk-thomas dirk-thomas added the question Further information is requested label Feb 3, 2020
@dirk-thomas
Copy link
Member

A new features is commonly not backported to already released distributions. The reason is a combination of effort as well as desire for stability / avoid risk of regressions. For that reason the change was not backported as described in ros/ros_comm#1205.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants