Always colorize Cygwin output

CHANGELOG.md

@@ -33,6 +33,8 @@ FEATURES:
 
 IMPROVEMENTS:
 
+  - Full Windows support in cmd.exe, PowerShell, Cygwin, and MingW based
+    environments.
   - By adding the "disabled" boolean flag to synced folders you can disable
     them altogether. [GH-1004]
   - Specify the default provider with the `VAGRANT_DEFAULT_PROVIDER`
@@ -45,6 +47,8 @@ BUG FIXES:
   - Don't preserve modified time when untarring boxes. [GH-1539]
   - Forwarded port auto-correct will not auto-correct to a port
     that is also in use.
+  - Cygwin will always output color by default. Specify `--no-color` to
+    override this.
 
 ## 1.1.6 (April 3, 2013)
 

bin/vagrant

@@ -22,17 +22,32 @@ $stderr.sync = true
 # environment.
 opts = {}
 
-# Disable color if the proper argument was passed or if we're
+# Disable color in a few cases:
-# on Windows since the default Windows terminal doesn't support
+#
-# colors.
+#   * --no-color is anywhere in our arguments
-if ARGV.include?("--no-color") || !$stdout.tty? || !Vagrant::Util::Platform.terminal_supports_colors?
+#   * STDOUT is not a TTY
-  # Delete the argument from the list so that it doesn't cause any
+#   * The terminal doesn't support colors (Windows)
-  # invalid arguments down the road.
+#
+if ARGV.include?("--no-color")
+  # Delete the argument from the list so that it doesn't
+  # cause any invalid arguments down the road.
   ARGV.delete("--no-color")
+
+  opts[:ui_class] = Vagrant::UI::Basic
+elsif !Vagrant::Util::Platform.terminal_supports_colors?
+  opts[:ui_class] = Vagrant::UI::Basic
+elsif !$stdout.tty?
+  # If we're not a TTY, verify we're not in Cygwin. Cygwin always
+  # reports that stdout is not a TTY, when in fact it is.
+  ENV["VAGRANT_DETECTED_OS"] ||= ""
+
+  if !ENV["VAGRANT_DETECTED_OS"].downcase.include?("cygwin")
     opts[:ui_class] = Vagrant::UI::Basic
-else
-  opts[:ui_class] = Vagrant::UI::Colored
   end
+end
+
+# Default to colored output
+opts[:ui_class] ||= Vagrant::UI::Colored
 
 # This is kind of hacky, and I'd love to find a better way to do this, but
 # if we're accessing the plugin interface, we want to NOT load plugins

lib/vagrant/util/platform.rb

@@ -67,7 +67,8 @@ module Vagrant
         # output.
         def terminal_supports_colors?
           if windows?
-            return ENV.has_key?("ANSICON")
+            return ENV.has_key?("ANSICON") ||
+              ENV["VAGRANT_DETECTED_OS"].downcase.include?("cygwin")
           end
 
           true