Skip to content

Commit df87b35

Browse files
committed
shelldriver: Redirect rx stderr to /dev/null
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>
1 parent 2437f0f commit df87b35

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

labgrid/driver/shelldriver.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,8 @@ def _get_xmodem_rx_cmd(self, filename):
320320
if not self._xmodem_cached_rx_cmd:
321321
if self._run('which rx')[0]:
322322
# busybox rx
323-
self._xmodem_cached_rx_cmd = "rx '{filename}'"
323+
# lrz may provide rx so redirect stderr for the same reason as below
324+
self._xmodem_cached_rx_cmd = "rx '{filename}' 2>/dev/null"
324325
elif self._run('which lrz')[0]:
325326
# redirect stderr to prevent lrz from printing "ready to receive
326327
# $file", which will confuse the XMODEM instance

0 commit comments

Comments
 (0)