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

3.8.0 regression with color support on ConEMU #294

Closed
bdw429s opened this issue Jul 9, 2018 · 7 comments
Closed

3.8.0 regression with color support on ConEMU #294

bdw429s opened this issue Jul 9, 2018 · 7 comments
Milestone

Comments

@bdw429s
Copy link
Contributor

bdw429s commented Jul 9, 2018

Upgrading from JLine 3.7.0 to 3.8.0 has a regression which now causes ConEMU to report as only supporting 8 colors instead of 256 colors. In both cases, org.jline.terminal.impl.jna.win.JnaWinSysTerminal is the terminal class in use.

Jline 3.7.0 running the system-colors command in CommandBox 4.1:
image

Jline 3.8.0 running the system-colors command in CommandBox 4.1:
image

@gnodet
Copy link
Member

gnodet commented Jul 17, 2018

Two questions:

  • is the behavior specific to Windows 10 ?
  • what is the type set on the terminal terminal.getType() ?

@bdw429s
Copy link
Contributor Author

bdw429s commented Jul 17, 2018

@gnodet Thanks for the reply.

is the behavior specific to Windows 10 ?

I haven't tried it on Windows 10. My main PC is Windows 7.
I just tried it on Windows 10 and I get the same behavior there as Windows 7.

what is the type set on the terminal terminal.getType() ?

I get null

@gnodet
Copy link
Member

gnodet commented Jul 17, 2018

Ok, I think null is the problem here, I'll investigate.

@gnodet gnodet closed this as completed Jul 17, 2018
@gnodet gnodet reopened this Jul 17, 2018
@gnodet
Copy link
Member

gnodet commented Jul 17, 2018

Could you try the following patch ?
https://github.com/jline/jline3/tree/issue-294

@bdw429s
Copy link
Contributor Author

bdw429s commented Jul 17, 2018

@gnodet I built the code on that branch and it seems to have fixed the regression. 256 colors are back in ConEMU. Thanks!

Any idea what version this will release in? I was hoping to include 3.8.0 in CommandBox due to some other fixes you had put in there for me but I'll need to wait until this fix going into a stable version.

@gnodet
Copy link
Member

gnodet commented Jul 17, 2018

JLine 3.8.2 should be available shortly in central.

@bdw429s
Copy link
Contributor Author

bdw429s commented Jul 17, 2018

Thanks again @gnodet !

@gnodet gnodet added this to the 3.8.2 milestone Jul 17, 2018
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

2 participants