Merge pull request #79 from chrissimpkins/ansi-color

Add ANSI color support and improved standard stream formatting

Author: spastorino

Cargo.lock

@@ -31,6 +31,7 @@ tee = "0.1"
 tempdir = "0.3.7"
 xz2 = "0.1.6"
 chrono = "0.4.11"
+colored="1.9"
 
 [dev-dependencies]
 quickcheck = "0.9.2"

Cargo.toml

@@ -15,6 +15,7 @@ use std::process::{self, Command};
 use std::str::FromStr;
 
 use chrono::{Date, DateTime, Duration, Utc};
+use colored::*;
 use failure::{bail, format_err, Fail, Error};
 use log::debug;
 use reqwest::blocking::Client;
@@ -634,7 +635,13 @@ fn print_results(cfg: &Config, client: &Client, bisection_result: &BisectionResu
         }
     }
 
-    eprintln!("regression in {}", toolchains[*found]);
+    let tc_found = format!("Regression in {}", toolchains[*found]);
+    eprintln!("");
+    eprintln!("");
+    eprintln!("{}", "*".repeat(80).dimmed().bold());
+    eprintln!("{}", tc_found.red());
+    eprintln!("{}", "*".repeat(80).dimmed().bold());
+    eprintln!("");
 }
 
 fn print_final_report(
@@ -653,14 +660,16 @@ fn print_final_report(
         ..
     } = ci_bisection_result;
 
-    eprintln!("");
-    eprintln!("");
-
-    eprintln!("==================================================================================");
-    eprintln!("= Please open an issue on Rust's github repository                               =");
-    eprintln!("= https://github.com/rust-lang/rust/issues/new                                   =");
-    eprintln!("= Below you will find a text that would serve as a starting point of your report =");
-    eprintln!("==================================================================================");
+    #[rustfmt::skip]
+    eprintln!("{}", "==================================================================================".dimmed());
+    #[rustfmt::skip]
+    eprintln!("{}", "= Please open an issue on Rust's github repository                               =".dimmed());
+    #[rustfmt::skip]
+    eprintln!("{}", "= https://github.com/rust-lang/rust/issues/new                                   =".dimmed());
+    #[rustfmt::skip]
+    eprintln!("{}", "= Below you will find a text that would serve as a starting point of your report =".dimmed());
+    #[rustfmt::skip]
+    eprintln!("{}", "==================================================================================".dimmed());
 
     eprintln!("");
 
@@ -1071,6 +1080,7 @@ fn bisect_ci_in_commits(
     }
 
     eprintln!("validated commits found, specifying toolchains");
+    eprintln!("");
 
     let toolchains = commits
         .into_iter()
@@ -1142,7 +1152,8 @@ fn main() {
         match err.downcast::<ExitError>() {
             Ok(ExitError(code)) => process::exit(code),
             Err(err) => {
-                eprintln!("ERROR: {}", err);
+                let error_str = "ERROR:".red().bold();
+                eprintln!("{} {}", error_str, err);
                 process::exit(1);
             }
         }

src/main.rs

@@ -12,6 +12,7 @@ use std::path::{Path, PathBuf};
 use std::process::{self, Command, Stdio};
 
 use chrono::{Date, naive, Utc};
+use colored::*;
 use dialoguer::Select;
 use failure::{Fail, Error};
 use flate2::read::GzDecoder;
@@ -139,7 +140,8 @@ impl Toolchain {
         client: &Client,
         dl_params: &DownloadParams,
     ) -> Result<(), InstallError> {
-        eprintln!("installing {}", self);
+        let tc_stdstream_str = format!("{}", self);
+        eprintln!("installing {}", tc_stdstream_str.green());
         let tmpdir = TempDir::new_in(&dl_params.tmp_dir, &self.rustup_name())
             .map_err(InstallError::TempDir)?;
         let dest = dl_params.install_dir.join(self.rustup_name());