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

os/PortCore: Do not set TOS to 0 if DSCP is an unknown string #2294

Merged
merged 1 commit into from
Jun 19, 2020

Conversation

drdanz
Copy link
Member

@drdanz drdanz commented Jun 17, 2020

Also improve debugging by adding a few debug messages

@drdanz drdanz self-assigned this Jun 17, 2020
@drdanz drdanz force-pushed the fix_qos_invalid_dscp branch 3 times, most recently from 2e54eed to bb3d27a Compare June 17, 2020 10:11
@drdanz drdanz added the PR Status: Continuous Integration - OK Continuous Integration for this PR passed (invalid if commits were added or modified after this) label Jun 19, 2020
@drdanz
Copy link
Member Author

drdanz commented Jun 19, 2020

Travis did not report the status back, but the build was successful

@drdanz

This comment has been minimized.

Also improve debugging by adding a few debug messages
@drdanz drdanz force-pushed the fix_qos_invalid_dscp branch from bb3d27a to 2fee002 Compare June 19, 2020 09:09
@drdanz drdanz merged commit 5c3406b into robotology:master Jun 19, 2020
@drdanz drdanz deleted the fix_qos_invalid_dscp branch June 19, 2020 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Library - YARP_os PR Status: Continuous Integration - OK Continuous Integration for this PR passed (invalid if commits were added or modified after this) PR Type: Bugfix This PR fixes some bug Resolution: Merged Target: YARP v3.4.0 Type: Breaking/Behaviour Change Involves breaking user code or behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant