Closed MartiUK closed 11 months ago
This update starting to give me too many The terminal does not have the capability to report the cursor position.
errors
@zouhair Could you provide an example, which I can test working on the proper patch?
Hmmm, it seems it's linked to smenu , may not be conemu bug, I use it in some script to generate simple menus that don't clear the console (as fzf does). I guess I have to use something else.
$ set -xv
history -a; history -n; history -a
++ history -a
++ history -n
++ history -a
(_z --add "$(command pwd -P 2>/dev/null)" 2>/dev/null &);
++ nonzero_return
+++ command pwd -P
++ RETVAL=0
++ '[' 0 -eq 0 ']'
++ echo ':)'
++ '[' 0 -ne 0 ']'
++ _z --add /home/poboxy
$ printf "Yes No Cancel" | smenu -d
printf "Yes No Cancel" | smenu -d
+ smenu -d
+ printf 'Yes No Cancel'
The terminal does not have the capability to report the cursor position.
history -a; history -n; history -a
++ history -a
++ history -n
++ history -a
(_z --add "$(command pwd -P 2>/dev/null)" 2>/dev/null &);
++ nonzero_return
++ RETVAL=1
++ '[' 1 -eq 0 ']'
++ '[' 1 -ne 0 ']'
++ echo 1
$ +++ command pwd -P
++ _z --add /home/poboxy
set +xv
set +xv
+ set +xv
$
This latest commit 230724 fixed it
Small coffee on the way 👍
UNSAFE_CONSOLE_REPORT_CHARS
does not cover everything, it's safer to not emit control characters in the title at all.