labgrid-project / labgrid

Embedded systems control library for development, testing and installation
https://labgrid.readthedocs.io/
Other
341 stars 177 forks source link

Xmodem executables autodetection fails #405

Closed bgermann closed 2 years ago

bgermann commented 5 years ago

commit 7521f159250 added XMODEM support for ShellDriver. Some systems (checked for Debian, Arch Linux, FreeBSD) rename the original lrzsz executable file names in their packages (removing the leading l) so that lrz and lsz are not available. rx is then wrongly detected as Busybox's rx and called with wrong arguments.

Checking for [l]sz respectively [l]rz and then for rx as Busybox version should do the trick for most systems.

Emantor commented 5 years ago

Sounds like a good solution to the problem.