Merge pull request #1178 from kbknapp/issue-1155

imp(Deps): doesnt compile ansi_term on Windows since its not used
clap-rs · Feb 13, 2018 · 5b373f5 · 5b373f5
2 parents 1e9e47e + 3bc5f4b
commit 5b373f5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -28,13 +28,15 @@ bitflags              = "1.0"
 unicode-width         = "0.1.4"
 textwrap              = "0.9.0"
 strsim    = { version = "0.7.0",  optional = true }
-ansi_term = { version = "0.10.0",  optional = true }
 yaml-rust = { version = "0.3.5",  optional = true }
 clippy    = { version = "~0.0.166", optional = true }
 atty      = { version = "0.2.2",  optional = true }
 vec_map   = { version = "0.8", optional = true }
 term_size = { version = "0.3.0", optional = true }
 
+[target.'cfg(not(windows))'.dependencies]
+ansi_term = { version = "0.10.0",  optional = true }
+
 [dev-dependencies]
 regex = "0.2"
 lazy_static = "1"

diff --git a/src/lib.rs b/src/lib.rs
@@ -528,7 +528,7 @@
 #![cfg_attr(feature = "lints", allow(doc_markdown))]
 #![cfg_attr(feature = "lints", allow(explicit_iter_loop))]
 
-#[cfg(feature = "color")]
+#[cfg(all(feature = "color", not(target_os = "windows")))]
 extern crate ansi_term;
 #[cfg(feature = "color")]
 extern crate atty;