fix: Don't depend on ansi_term in windows to make windows 7 work

Fixes #777
gluon-lang · Aug 30, 2019 · 58e2a8b · 58e2a8b
1 parent b910eea
commit 58e2a8b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/repl/Cargo.toml b/repl/Cargo.toml
@@ -27,7 +27,6 @@ gluon_codegen = { path = "../codegen", version = "0.12.0" } # GLUON
 gluon_format = { version = "0.12.0", path = "../format" } # GLUON
 gluon_doc = { version = "0.12.0", path = "../doc" } # GLUON
 
-ansi_term = "0.12"
 app_dirs = "1.0.0"
 futures = "0.1.11"
 futures-cpupool = "0.1"
@@ -48,6 +47,9 @@ codespan-reporting = "0.3"
 serde = "1"
 serde_derive = "1"
 
+[target.'cfg(not(windows))'.dependencies]
+ansi_term = "0.12"
+
 [dev-dependencies]
 pretty_assertions = "0.6"
 

diff --git a/repl/src/repl.rs b/repl/src/repl.rs
@@ -190,9 +190,12 @@ impl rustyline::highlight::Highlighter for Completer {
 
     fn highlight_hint<'h>(&self, hint: &'h str) -> Cow<'h, str> {
         // TODO Detect when windows supports ANSI escapes
-        if cfg!(windows) {
+        #[cfg(windows)]
+        {
             Cow::Borrowed(hint)
-        } else {
+        }
+        #[cfg(not(windows))]
+        {
             use ansi_term::Style;
             Cow::Owned(Style::new().dimmed().paint(hint).to_string())
         }