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

Use is_unsupported_term on both unix and windows #834

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

gwenn
Copy link
Collaborator

@gwenn gwenn commented Dec 14, 2024

Fix #832

@gwenn
Copy link
Collaborator Author

gwenn commented Dec 16, 2024

At least no regression with Powershell.

@jasagredo
Copy link

Unfortunately Scryer prolog is using rustyline 14.0.0 and your changes do not cherry-pick cleanly there. I can confirm that it works with the following patch I crafted when I opened the issue:
windows-term.patch

The changes are fairly similar to those in your branch, but yours look more organized (mine have duplicated UNSUPPORTED_TERMS and so on) so I'm pretty sure it will work, albeit I have not really checked it.

@gwenn gwenn merged commit 5910f9e into kkawakam:master Dec 17, 2024
3 checks passed
@gwenn gwenn deleted the win_term branch December 17, 2024 06:30
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

Successfully merging this pull request may close these issues.

Send prompt to stdout even if stdin is not a tty or check TERM also on Windows
2 participants