Support Windows in extra::term

[catamorphism]

Do it. Do it now. (Or at least sometime.)

[emberian]

This is easier to do now that it has been refactored to support 8-color terminals and handle terminfo properly. Aetherspawn in IRC started work on this.

[emberian]

Updated issue title to reflect std->extra

[vadimcn]

Windows console does not support escape codes. In order to change colors one must use console API functions, such as SetConsoleTextAttribute. Unfortunately these require console handle, but there no obvious way to extract a file handle from io::Writer. I suppose we could implement Terminal<StdWriter> instead of Terminal<Writer> and add a get_fd() method to the former...

@alexcrichton: any other ideas?

[jhasse]

Hi! I tried to fix this and noticed that the msys terminal supports ANSI colors but msys doesn't come with terminfo. So this patch tries to figure out if it is run from a msys terminal (env var TERM is cygwin, since msys is a modified fork of an older cygwin version) and then enables colors.

Here is a screenshot of my patch in action: https://www.dropbox.com/s/mi7gxp94ilkis60/Screenshot%202013-12-17%2008.39.48.png

Strangely this also works with cmd.exe on my system. Maybe because I have msysgit installed? I don't know how to detect this though: https://www.dropbox.com/s/tvk5hd3n8ur4cnd/Screenshot%202013-12-17%2008.42.09.png

Sorry if there are issues in my patch, I'm very new to Rust and unfamiliar with the language.

[klutzy]

@jhasse rust's stdio uses libuv's tty interface. libuv implements some ANSI color parser, so I guess that extra::term worked via libuv, not via msys support.

[jhasse]

@klutzy Cool, so this means it should work on all Windows systems out of the box, right?

[emberian]

We only use libuv in M:N mode, with libgreen. We shouldn't depend on the
specific runtime in extra::term.

On Tue, Dec 17, 2013 at 3:36 AM, Jan Niklas Hasse
notifications@github.comwrote:

@klutzy https://github.com/klutzy Cool, so this means it should work on
all Windows systems out of the box, right?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/2807#issuecomment-30733941
.

[jhasse]

Okay I've reverted the change for cmd.exe. This new patch should now only change something for the msys terminal on windows.