Skip to content

shelldriver: Redirect rx stderr to /dev/null#892

Merged
jluebbe merged 1 commit intolabgrid-project:masterfrom
JoshuaWatt:rx-stderr
Feb 18, 2022
Merged

shelldriver: Redirect rx stderr to /dev/null#892
jluebbe merged 1 commit intolabgrid-project:masterfrom
JoshuaWatt:rx-stderr

Conversation

@JoshuaWatt
Copy link
Copy Markdown
Contributor

lrzsz also provides the rx command, and it will output the same banner
to stderr as the lrz command. Suppress stderr to handle this case

Signed-off-by: Joshua Watt Joshua.Watt@garmin.com

Description

Checklist

  • Documentation for the feature
  • Tests for the feature
  • The arguments and description in doc/configuration.rst have been updated
  • Add a section on how to use the feature to doc/usage.rst
  • Add a section on how to use the feature to doc/development.rst
  • CHANGES.rst has been updated
  • PR has been tested
  • Man pages have been regenerated

lrzsz also provides the `rx` command, and it will output the same banner
to stderr as the `lrz` command. Suppress stderr to handle this case

Signed-off-by: Joshua Watt <Joshua.Watt@garmin.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 17, 2022

Codecov Report

Merging #892 (df87b35) into master (2437f0f) will not change coverage.
The diff coverage is 0.0%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #892   +/-   ##
======================================
  Coverage    56.9%   56.9%           
======================================
  Files         149     149           
  Lines       11011   11011           
======================================
  Hits         6268    6268           
  Misses       4743    4743           
Impacted Files Coverage Δ
labgrid/driver/shelldriver.py 28.0% <0.0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2437f0f...df87b35. Read the comment docs.

@jluebbe jluebbe merged commit 72494bc into labgrid-project:master Feb 18, 2022
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.

3 participants