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 detection failing #26

Closed
Vampire opened this issue Aug 29, 2017 · 5 comments
Closed

Terminal detection failing #26

Vampire opened this issue Aug 29, 2017 · 5 comments

Comments

@Vampire
Copy link

Vampire commented Aug 29, 2017

When executing Gradle on Git Bash from Git for Windows, or from ConEmus conemu-cyg-32.exe (which then starts the Cygwin Bash), no progress and colors are shown, because https://github.com/adammurdoch/native-platform/blob/master/src/main/cpp/win.cpp#L404-L419 returns false. With plain Cygwin Bash it works fine though.

@Vampire
Copy link
Author

Vampire commented Aug 30, 2017

At least for the ConEmu Cygwin connector it seems that the detection mechanism used here is wrong, as the handles are not WinAPI capable, but "true" posic tty handles. So I guess the detection has to be changed to handle this properly.

@adammurdoch
Copy link
Contributor

The terminal handling can now detect Mintty for Cygwin and msys. Seems to be working fine with Git Bash from Git for Windows now. The fix will make its way into a Gradle release soon.

@sschuberth
Copy link

sschuberth commented Apr 15, 2019

However, terminal width detection does not seem to be properly working with MinTTY / Bash from Git for Windows. If I resize the window to > 80 columns, Gradle's output is still cropped to 80 chars per line.

@adammurdoch
Copy link
Contributor

Yes, it currently only detects whether the terminal is there. It does not yet know how to determine the terminal size.

@adammurdoch
Copy link
Contributor

I've added #33 for the terminal size issue. I'm going to close this issue, as the changes to detect the terminal are now in Gradle (as of Gradle 5.2).

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

3 participants