k3ng / k3ng_cw_keyer

K3NG Arduino CW Keyer

Home Page:http://blog.radioartisan.com/arduino-cw-keyer/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

extended cli not working on linux (with solution)

devcpu opened this issue · comments

All extended cli command are not working on linux because of the different lineend (\r vs. \n).
In function cli_extended_commands (line ~ 12884)

please change

if (incoming_serial_byte == 13) {   // carriage return - get out

to

if (incoming_serial_byte == 13 || incoming_serial_byte == 10) {   // carriage return - get out

that's it.

Can't make a pull request, but it is quite simple.

BTW. Many thanks for the software.
73 de DL7UXA

The terminal in arduino ide.
I found a lot of similar code. Some check of 10 and 13, some not. I see this because same problem programing memories from CLI.

commented

It think its strange cause windows end of line should be "\r\n" and the unix one "\n". https://stackoverflow.com/questions/1761051/difference-between-n-and-r
please correct me if I"m wrong

Fixed in version 2020.07.26.01

Thanks!
73
Goody
K3NG