Terminal left in broken state

[dwillmore]

After exiting from radar, any mouse action in the text terminal from which it was run now disaplay as excape characters. This persists until the terminal is closed. ESC c did not clear this conditon. Seems radar is not restoring the terminal condition upon exit.

[wcampbell0x2a]

After exiting from radar, any mouse action in the text terminal from which it was run now disaplay as excape characters. This persists until the terminal is closed. ESC c did not clear this conditon. Seems radar is not restoring the terminal condition upon exit.

What terminal are you using, I'll test this later.

[dwillmore]

Xfce terminal under Fedora 34. Should behave like a normal xterm in this context. Thank you!

…

On Thu, Feb 17, 2022, 11:03 AM wcampbell ***@***.***> wrote: After exiting from radar, any mouse action in the text terminal from which it was run now disaplay as excape characters. This persists until the terminal is closed. ESC c did not clear this conditon. Seems radar is not restoring the terminal condition upon exit. What terminal are you using, I'll test this later. — Reply to this email directly, view it on GitHub <#116 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACPEX7CF7RWGGSEU7PI3BLLU3UL6BANCNFSM5OU22YHA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[dwillmore]

I tried it with the Gnome terminal and got similar results.

[wcampbell0x2a]

I can't reproduce this with gnome-terminal, did you exit radar after a panic or something or were you using q or ctrl+c?

[dwillmore]

I used 'q'.

…

On Thu, Feb 17, 2022, 7:31 PM wcampbell ***@***.***> wrote: I can't reproduce this with gnome-terminal, did you exit radar after a panic or something or were you using q or ctrl+c? — Reply to this email directly, view it on GitHub <#116 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACPEX7DVB5AHG37Y7AHYHRDU3WHMXANCNFSM5OU22YHA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[wcampbell0x2a]

Huh, I've run into what you are talking about randomly. I'll try and reproduce it more..

[dwillmore]

Thank you. It is every time for me. It even happens some times when I get just the error message.

…

On Thu, Feb 17, 2022, 7:36 PM wcampbell ***@***.***> wrote: Huh, I've run into what you are talking about randomly. I'll try and reproduce it more.. — Reply to this email directly, view it on GitHub <#116 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACPEX7FPAY4KYUYPB4ID2I3U3WICTANCNFSM5OU22YHA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[wcampbell0x2a]

Can you confirm that this merge request: #124, fixed your issue? Thanks

[dwillmore]

I can't build here due to the package requiring a newer version of Rust than Fedora 34 provides. I can test a binary, but there's no release with this change in it, yet.

[wcampbell0x2a]

Sorry about that, I always want to use new features. I usually use rustup so that's not a problem.

That being said, you can grab the latest binaries from the CI build like such: https://github.com/rsadsb/adsb_deku/actions/runs/1913359381

[dwillmore]

I will try that, thank you.

[dwillmore]

I can verify that this fix works. Thank you. I had to rig up rtl_adsb and netcat because 1090 required a newer version of SoapySDR, but that shouldn't invalidate the testing. I reproduced the old failure and the new success just to be sure.

Thank you, again!