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

brace matching broken in iTerm #119

Closed
swannodette opened this issue Aug 25, 2015 · 9 comments
Closed

brace matching broken in iTerm #119

swannodette opened this issue Aug 25, 2015 · 9 comments

Comments

@swannodette
Copy link

In iTerm, https://www.iterm2.com, I get noise characters instead of brace matching.

@mfikes
Copy link
Member

mfikes commented Aug 25, 2015

I'll look into it. In the meanwhile, the VT100 terminal control characters can be turned off by passing -d (or --dumb-terminal) to Planck. (This option is also useful if you'd like to use rlwrap with Planck.)

@mfikes mfikes closed this as completed in 72fb33b Aug 25, 2015
@eggsyntax
Copy link

No reason to reopen the ticket, but for anyone else who has this issue and lands here, updating to the iTerm2 v3 beta also fixes it.

@davidhq
Copy link

davidhq commented Sep 22, 2015

-d is not useful because it also disables moving around with arrows...

I'm reluctant to upgrade to iTerm 3 beta... but... is it stable? Anyone using it?

When will this be working for iTerm 2, will it?

thank you

@mfikes
Copy link
Member

mfikes commented Sep 22, 2015

@davidhq If you are able to build from master, then you can get this fix before it is released. To do this using brew simply entails

brew install --HEAD planck

@mfikes
Copy link
Member

mfikes commented Sep 22, 2015

Or you can use Planck 1.6 in iTerm with rlwrap via

rlwrap planck -d

and at least have line editing and history.

@davidhq
Copy link

davidhq commented Sep 22, 2015

Thank you, bulding from master worked!

mfikes added a commit that referenced this issue Oct 17, 2015
Was placing negative integer into CUB (cursor
backwards) and similarly into CUF (cursor
forwards) upon restore. Fixes #119


Former-commit-id: 72fb33b
@attentive
Copy link

Quick confirmation for anyone else passing—was seeing a variant of this issue with noise characters in the repl and updating Planck to 1.9 with

brew install --HEAD planck

resolved it.

@mfikes
Copy link
Member

mfikes commented Nov 24, 2015

@attentive This was fixed in Planck 1.7, so there should be no need to run a custom build from HEAD. You probably just need to do a brew update.

@attentive
Copy link

Thanks Mike—all seems good so I'll leave it for now.

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

5 participants