Skip to content
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

tcell panic on TERM=xterm-termite #391

Closed
jangler opened this issue Oct 3, 2020 · 6 comments
Closed

tcell panic on TERM=xterm-termite #391

jangler opened this issue Oct 3, 2020 · 6 comments

Comments

@jangler
Copy link

jangler commented Oct 3, 2020

Any plans to fix this? xterm-termite is what my termite uses by default, and tcell panics on it:

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0xa48 pc=0x72cf42]

goroutine 1 [running]:
github.com/gdamore/tcell/terminfo.LookupTerminfo(0xc000018105, 0xd, 0xc000018105, 0xd, 0xcbe9207f64715fde)
	/home/brandon/src/github.com/gdamore/tcell/terminfo/terminfo.go:752 +0x102
github.com/gdamore/tcell.NewTerminfoScreen(0x7d2d01, 0x0, 0x30, 0x50)
	/home/brandon/src/github.com/gdamore/tcell/tscreen.go:44 +0x60
github.com/gdamore/tcell.NewScreen(0x7d71a0, 0xbc14c0, 0x7f4c71ec3108, 0x0)
	/home/brandon/src/github.com/gdamore/tcell/screen.go:208 +0x22
...

I see that xterm-termite had a terminfo added in 2efc922, but I guess it was lost in the fix to #301? TERM=termite works fine.

jangler added a commit to jangler/oracles-randomizer that referenced this issue Oct 3, 2020
termbox is no longer maintained and was causing problems for Windows
users. Unfortunately tcell panics on TERM=xterm-termite, although
TERM=termite, TERM=xterm, and TERM=screen work fine.

See gdamore/tcell#391
@halturin
Copy link

halturin commented Oct 8, 2020

the same issue with tmux-256colors

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0xa48 pc=0x50eecf]

goroutine 1 [running]:
github.com/gdamore/tcell/terminfo.LookupTerminfo(0xc000018165, 0xd, 0xc000018165, 0xd, 0x63ac60)
        /home/taras/devel/goenv/src/github.com/gdamore/tcell/terminfo/terminfo.go:752 +0x10f
github.com/gdamore/tcell.NewTerminfoScreen(0xc00013dd90, 0x7bbf10, 0x64a960, 0xc00013dd90)
        /home/taras/devel/goenv/src/github.com/gdamore/tcell/tscreen.go:44 +0x65
github.com/gdamore/tcell.NewScreen(0x8, 0x63ac60, 0x1, 0x0)
        /home/taras/devel/goenv/src/github.com/gdamore/tcell/screen.go:208 +0x25
main.run()
        /home/taras/devel/goenv/src/github.com/gokcehan/lf/client.go:19 +0x4e
main.main()
        /home/taras/devel/goenv/src/github.com/gokcehan/lf/main.go:269 +0xbb8
❯❯❯❯ echo $TERM                                                                                                                                                                         ~/D/development
tmux-256color

@gdamore
Copy link
Owner

gdamore commented Oct 8, 2020

I will fix, I just haven't had enough time to look at it yet. Probably in the next day or two.

@gdamore
Copy link
Owner

gdamore commented Oct 14, 2020

There are two problems here.

  1. The crash is due to a big mishandling the non-existent terminal description.
  2. The terminal descriptions that should be there aren't.

Stay tuned.

@gdamore
Copy link
Owner

gdamore commented Oct 14, 2020

tmux-256color must not be defined in your terminfo -- likewise xterm-termite.

They are not in my terminfo, and that's probably part of the problem.

@halturin
Copy link

without tmux-256color italic font doesn't work in my vim. my default TERM is xterm-256color (outside of the tmux) and everything works well. I would appreciate if you give me some advice or workaround to fix this issue without losing italic lettering

@gdamore
Copy link
Owner

gdamore commented Oct 14, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants