Auto merge of #1189 - michaelfletchercgy:show-when-active-not-installed, r=alexcrichton

Fix 'show' so it displays helpful information if the active toolchain is not installed.

Fixed 'show' so that it displays helpful information if the active toolchain is not installed.

```
$ export RUSTUP_TOOLCHAIN=beta
$ rustup show
Default host: x86_64-unknown-linux-gnu
installed toolchains
--------------------

stable-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu

active toolchain
----------------

(error: override toolchain 'beta' is not installed, the RUSTUP_TOOLCHAIN environment variable specifies an uninstalled toolchain)
```

Fixes #1178

Author: bors

src/rustup-cli/rustup_mode.rs

@@ -10,6 +10,7 @@ use self_update;
 use std::path::Path;
 use std::process::Command;
 use std::iter;
+use std::error::Error;
 use term2;
 use std::io::{self, Write};
 use help::*;
@@ -496,15 +497,21 @@ fn show(cfg: &Cfg) -> Result<()> {
 
     let ref cwd = try!(utils::current_dir());
     let installed_toolchains = try!(cfg.list_toolchains());
-    let active_toolchain = try!(cfg.find_override_toolchain_or_default(cwd));
-    let active_targets = if let Some((ref t, _)) = active_toolchain {
-        match t.list_components() {
-            Ok(cs_vec) => cs_vec
-                .into_iter()
-                .filter(|c| c.component.pkg == "rust-std")
-                .filter(|c| c.installed)
-                .collect(),
-            Err(_) => vec![]
+    let active_toolchain = cfg.find_override_toolchain_or_default(cwd);
+
+    // active_toolchain will carry the reason we don't have one in its detail.
+    let active_targets = if let Ok(ref at) = active_toolchain {
+        if let Some((ref t, _)) = *at {
+            match t.list_components() {
+                Ok(cs_vec) => cs_vec
+                    .into_iter()
+                    .filter(|c| c.component.pkg == "rust-std")
+                    .filter(|c| c.installed)
+                    .collect(),
+                Err(_) => vec![]
+            }
+        } else {
+            vec![]
         }
     } else {
         vec![]
@@ -523,9 +530,7 @@ fn show(cfg: &Cfg) -> Result<()> {
 
     if show_installed_toolchains {
         if show_headers { print_header("installed toolchains") }
-        let default = try!(cfg.find_default());
-        let default_name = default.map(|t| t.name().to_string())
-                           .unwrap_or("".into());
+        let default_name = try!(cfg.get_default());
         for t in installed_toolchains {
             if default_name == t {
                 println!("{} (default)", t);
@@ -550,16 +555,27 @@ fn show(cfg: &Cfg) -> Result<()> {
         if show_headers { print_header("active toolchain") }
 
         match active_toolchain {
-            Some((ref toolchain, Some(ref reason))) => {
-                println!("{} ({})", toolchain.name(), reason);
-                println!("{}", common::rustc_version(toolchain));
-            }
-            Some((ref toolchain, None)) => {
-                println!("{} (default)", toolchain.name());
-                println!("{}", common::rustc_version(toolchain));
+            Ok(atc) => {
+                match atc {
+                    Some((ref toolchain, Some(ref reason))) => {
+                        println!("{} ({})", toolchain.name(), reason);
+                        println!("{}", common::rustc_version(toolchain));
+                    }
+                    Some((ref toolchain, None)) => {
+                        println!("{} (default)", toolchain.name());
+                        println!("{}", common::rustc_version(toolchain));
+                    }
+                    None => {
+                        println!("no active toolchain");
+                    }
+                }
             }
-            None => {
-                println!("no active toolchain");
+            Err(err) => {
+                if let Some(cause) = err.cause() {
+                    println!("(error: {}, {})", err, cause);
+                } else {
+                    println!("(error: {})", err);
+                }
             }
         }
 

src/rustup/config.rs

@@ -326,6 +326,14 @@ impl Cfg {
         })
     }
 
+    pub fn get_default(&self) -> Result<String> {
+        self.settings_file.with(|s| { 
+            Ok(s.default_toolchain.clone().unwrap())
+        })
+    }
+
+
+
     pub fn list_toolchains(&self) -> Result<Vec<String>> {
         if utils::is_directory(&self.toolchains_dir) {
             let mut toolchains: Vec<_> = try!(utils::read_dir("toolchains", &self.toolchains_dir))

tests/cli-rustup.rs

@@ -4,6 +4,7 @@ extern crate rustup_dist;
 extern crate rustup_utils;
 extern crate rustup_mock;
 extern crate tempdir;
+extern crate regex;
 
 use std::fs;
 use std::env::consts::EXE_SUFFIX;
@@ -15,6 +16,7 @@ use rustup_mock::clitools::{self, Config, Scenario,
                             expect_err,
                             set_current_dist_date,
                             this_host_triple};
+use regex::Regex;
 
 macro_rules! for_host { ($s: expr) => (&format!($s, this_host_triple())) }
 
@@ -624,11 +626,10 @@ fn show_toolchain_override_not_installed() {
         let mut cmd = clitools::cmd(config, "rustup", &["show"]);
         clitools::env(config, &mut cmd);
         let out = cmd.output().unwrap();
-        assert!(!out.status.success());
-        let stderr = String::from_utf8(out.stderr).unwrap();
-        assert!(stderr.starts_with(
-                for_host!("error: override toolchain 'nightly-{0}' is not installed")));
-        assert!(!stderr.contains("info: caused by: not a directory: "));
+        assert!(out.status.success());
+        let stdout = String::from_utf8(out.stdout).unwrap();
+        assert!(!stdout.contains("not a directory"));
+        assert!(Regex::new(r"error: override toolchain 'nightly.*' is not installed, the directory override for '.*' specifies an uninstalled toolchain").unwrap().is_match(&stdout))
     });
 }
 
@@ -659,9 +660,9 @@ fn show_toolchain_env_not_installed() {
         let out = cmd.output().unwrap();
         // I'm not sure this should really be erroring when the toolchain
         // is not installed; just capturing the behavior.
-        assert!(!out.status.success());
-        let stderr = String::from_utf8(out.stderr).unwrap();
-        assert!(stderr.starts_with("error: override toolchain 'nightly' is not installed\n"));
+        assert!(out.status.success());
+        let stdout = String::from_utf8(out.stdout).unwrap();
+        assert!(stdout.contains("override toolchain 'nightly' is not installed, the RUSTUP_TOOLCHAIN environment variable specifies an uninstalled toolchain"));
     });
 }