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

terminal broken after exiting tmux #64

Closed
ysangkok opened this issue Apr 27, 2015 · 6 comments
Closed

terminal broken after exiting tmux #64

ysangkok opened this issue Apr 27, 2015 · 6 comments
Assignees
Labels

Comments

@ysangkok
Copy link
Collaborator

reproduction:

  1. launch jor1k
  2. start tmux
  3. press ctrl-d
  4. terminal broken (only one line visible, even reset or stty sane doesn't fix it)
@s-macke s-macke added the bug label Apr 27, 2015
@s-macke s-macke self-assigned this Apr 27, 2015
@s-macke
Copy link
Owner

s-macke commented Apr 27, 2015

Yes, there is some terminal weirdness. But the terminal, which is used has no internal states, which could alter the \r \n behavior. That means, that tmux alters somehow the assumptions about the terminal.
I havnt's been into the details yet. But I think, this has to do with some missing terminal features. Look at "stty -a". He doesn't tell me the rows and columns and there is even a "No such file or directory" error. I guess when I fix these errors, tmux will also work. The problem is, I don't know how. I guess I have to do some digging in the kernel code.
I don't think, that another Javascript terminal fixes this behavior. But I haven't tried it.

@s-macke
Copy link
Owner

s-macke commented Apr 27, 2015

The "stty -a" output is identical before and after tmux.

@ZaDarkSide
Copy link

Try http://terminal.jcubic.pl/

@s-macke
Copy link
Owner

s-macke commented Apr 28, 2015

If you set "stty rows 24" before you start tmux the problem disappears.
The question is how to solve it. When terminals change their size they send the new rows and columns. But do they also send the data when they initialize? What is the correct behavior? I can set the rows in the init script. But I don't like the solution, because jor1k allows other terminals with other sizes too.

@s-macke
Copy link
Owner

s-macke commented May 18, 2015

87c6b04
Can you check if it works?

@ysangkok
Copy link
Collaborator Author

works great, thanks

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

No branches or pull requests

3 participants