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

Add tty detection for MSYS terminals. #12

Merged
merged 1 commit into from
Jan 14, 2017

Conversation

BurntSushi
Copy link
Contributor

This improves the tty detection in MSYS terminal environments on
Windows. Specifically, it uses heuristics to determine whether a
standard handle is a pipe or a tty.

Behavior in normal Windows consoles remains unchanged.

Fixes #10

@BurntSushi
Copy link
Contributor Author

We should try to get #11 merged first. Once that's done, I'll rebase this PR. :-)

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 1c423c3 on BurntSushi:mintty into ** on softprops:master**.

1 similar comment
@coveralls
Copy link

coveralls commented Jan 8, 2017

Coverage Status

Changes Unknown when pulling 1c423c3 on BurntSushi:mintty into ** on softprops:master**.

@BurntSushi
Copy link
Contributor Author

@softprops Thanks! If CI passes, then I think this one should be good to go too. :-)

FWIW, I realize this implementation is a bit gnarly. I'd be happy to play unofficial maintainer. I'm watching this repo, so I'll keep an eye on it if any bugs crop up.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 818dda7 on BurntSushi:mintty into ca2ab17 on softprops:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 818dda7 on BurntSushi:mintty into ca2ab17 on softprops:master.

This improves the tty detection in MSYS terminal environments on
Windows. Specifically, it uses heuristics to determine whether a
standard handle is a pipe or a tty.

Behavior in normal Windows consoles remains unchanged.

Fixes softprops#10
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling db8d55f on BurntSushi:mintty into ca2ab17 on softprops:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling db8d55f on BurntSushi:mintty into ca2ab17 on softprops:master.

@softprops softprops merged commit 1f727b1 into softprops:master Jan 14, 2017
@BurntSushi BurntSushi deleted the mintty branch January 14, 2017 18:22
@BurntSushi
Copy link
Contributor Author

@softprops Thanks! Sorry to be a bother, but could you also publish a new release too?

@softprops
Copy link
Owner

done

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.

3 participants