k3ng / k3ng_cw_keyer

K3NG Arduino CW Keyer
http://blog.radioartisan.com/arduino-cw-keyer/
GNU General Public License v3.0
416 stars 216 forks source link

extended cli not working on linux (with solution) #101

Closed devcpu closed 4 years ago

devcpu commented 4 years ago

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

k3ng commented 4 years ago

What terminal program are you using?

On Sat, Jul 25, 2020 at 6:06 PM Johannes Arlt notifications@github.com wrote:

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

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/k3ng/k3ng_cw_keyer/issues/101, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZNU4EVAFVNR6JK6OFG3BTR5NJNTANCNFSM4PHU7Q4A .

devcpu commented 4 years ago

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.

k3ng commented 4 years ago

Have you tried to set your Arduino IDE terminal to "Carriage Return" using the control at the bottom of the window?

On Sat, Jul 25, 2020 at 9:15 PM Johannes Arlt notifications@github.com wrote:

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.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/k3ng/k3ng_cw_keyer/issues/101#issuecomment-663923390, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZNU4A5FF2NPBP76NWVL4DR5N7RPANCNFSM4PHU7Q4A .

tuxun commented 4 years ago

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

k3ng commented 4 years ago

Fixed in version 2020.07.26.01

Thanks! 73 Goody K3NG