-
Notifications
You must be signed in to change notification settings - Fork 373
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Weird characters with ncurses 6.1 #185
Comments
Are you using |
Normally yes, but I did the test without tmux. |
Tested with Hyper, gnome-terminal, even with disabling all .profile&zshrc. (all broken) Did a diff of "set" between gnome-terminal and xterm, found a workaround: TERM=xterm #works!!! Seems like the bug only gets triggered with TERM=xterm-256color |
Downgrading to For anyone on Arch Linux, I've created a PKGBUILD file that gets you going quickly: |
Same here with peco & opensuse tumbleweed. |
Confirmed I am seeing this bug as well with ncurses 6.1 on Arch. It will effect anything using this lib including the many popular tools built on termui.
|
I'll have a look at it this weekend. Hopefully won't forget about it. |
Should work now. Ncurses 6.1 slightly altered the terminfo file format. Quoting man page:
Although doc implies octal number (and original magic number is also specified in octal). What I see in terminfo files is just a little endian 16-bit number 542. Whatever... |
It seems there's an issue with termbox-go (which termui depends on) with ncurses 6. Apply workaround of setting TERM=xterm mentioned in nsf/termbox-go#185
Documentation says it should have "a different magic number (0542)" (quoting "man 5 term"). What I see in practice however is a plain number 542. When this number is present, numbers array is an array of signed 32 bit integers instead of 16 bit integers. Take that into account when skipping it. Fixes nsf#185.
Peco 0.5.3 included an update to the termbox-go dependency to fix a bug, this change did not make it into deps.nix. See: - peco/peco#446 - peco/peco#447 - nsf/termbox-go#185
Peco 0.5.3 included an update to the termbox-go dependency to fix a bug, this change did not make it into deps.nix. See: - peco/peco#446 - peco/peco#447 - nsf/termbox-go#185
After upgrading ncurses to v6.1 weird characters appear on the screen when I try to use peco.
Thanks to @mattn I learned that peco uses termbox-go under the hood
and that the issue might happen because terminfo was also modified with the latest ncurses upgrade.
infocmp diff:
ncurses version: 6.1.20180127
TERM: xterm-256color
OS: Arch Linux
Linux kernel: 4.14.20-1-lts, x86_64
peco/peco#446
The text was updated successfully, but these errors were encountered: