Merge branch 'master' into db-bump-version

# Conflicts:
#	Cargo.lock

Author: Diggsey

Cargo.lock

@@ -35,21 +35,21 @@ rustup-dist = { path = "src/rustup-dist" }
 rustup-utils = { path = "src/rustup-utils" }
 download = { path = "src/download" }
 clap = "2.18.0"
-error-chain = "0.11"
+error-chain = "0.12.0"
 itertools = "0.7.6"
 libc = "0.2.0"
 markdown = "0.2"
-rand = "0.4.2"
-regex = "0.2"
-remove_dir_all = "0.3.0"
+rand = "0.5.2"
+regex = "1.0.1"
+remove_dir_all = "0.5.1"
 same-file = "1.0"
 scopeguard = "0.3"
 serde = "1.0"
 serde_derive = "1.0"
 serde_json = "1.0"
 sha2 = "0.7.0"
 tempdir = "0.3.4"
-tempfile = "2.1.4"
+tempfile = "3.0.2"
 term = "0.5.1"
 time = "0.1.34"
 toml = "0.4"

Cargo.toml

@@ -14,7 +14,7 @@ curl-backend = ["curl"]
 reqwest-backend = ["reqwest", "env_proxy", "lazy_static"]
 
 [dependencies]
-error-chain = "0.11"
+error-chain = "0.12"
 url = "1.2"
 curl = { version = "0.4.6", optional = true }
 env_proxy = { version = "0.2.0", optional = true }

src/download/Cargo.toml

@@ -17,6 +17,15 @@ pub static SHOW_HELP: &'static str = r"DISCUSSION:
     If there are multiple toolchains installed then all installed
     toolchains are listed as well.";
 
+pub static SHOW_ACTIVE_TOOLCHAIN_HELP: &'static str = r"DISCUSSION:
+    Shows the name of the active toolchain.
+
+    This is useful for figuring out the active tool chain from
+    scripts.
+
+    You should use `rustc --print sysroot` to get the sysroot, or
+    `rustc --version` to get the toolchain version.";
+
 pub static UPDATE_HELP: &'static str = r"DISCUSSION:
     With no toolchain specified, the `update` command updates each of
     the installed toolchains from the official release channels, then

src/rustup-cli/help.rs

@@ -29,7 +29,10 @@ pub fn main() -> Result<()> {
     cfg.check_metadata_version()?;
 
     match matches.subcommand() {
-        ("show", Some(_)) => show(cfg)?,
+        ("show", Some(c)) => match c.subcommand() {
+            ("active-toolchain", Some(_)) => show_active_toolchain(cfg)?,
+            (_, _) => show(cfg)?
+        },
         ("install", Some(m)) => update(cfg, m)?,
         ("update", Some(m)) => update(cfg, m)?,
         ("uninstall", Some(m)) => toolchain_remove(cfg, m)?,
@@ -110,7 +113,13 @@ pub fn cli() -> App<'static, 'static> {
         .subcommand(
             SubCommand::with_name("show")
                 .about("Show the active and installed toolchains")
-                .after_help(SHOW_HELP),
+                .after_help(SHOW_HELP)
+                .setting(AppSettings::VersionlessSubcommands)
+                .setting(AppSettings::DeriveDisplayOrder)
+                .subcommand(SubCommand::with_name("active-toolchain")
+                    .about("Show the active toolchain")
+                    .after_help(SHOW_ACTIVE_TOOLCHAIN_HELP)
+                ),
         )
         .subcommand(
             SubCommand::with_name("install")
@@ -360,6 +369,11 @@ pub fn cli() -> App<'static, 'static> {
                 .alias("docs")
                 .about("Open the documentation for the current toolchain")
                 .after_help(DOC_HELP)
+                .arg(
+                    Arg::with_name("path")
+                        .long("path")
+                        .help("Only print the path to the documentation"),
+                )
                 .arg(
                     Arg::with_name("book")
                         .long("book")
@@ -733,6 +747,19 @@ fn show(cfg: &Cfg) -> Result<()> {
     Ok(())
 }
 
+fn show_active_toolchain(cfg: &Cfg) -> Result<()> {
+    let ref cwd = utils::current_dir()?;
+    match cfg.find_override_toolchain_or_default(cwd)? {
+        Some((ref toolchain, _)) => {
+            println!("{}", toolchain.name());
+        },
+        None => {
+            // Print nothing
+        }
+    }
+    Ok(())
+}
+
 fn target_list(cfg: &Cfg, m: &ArgMatches) -> Result<()> {
     let toolchain = explicit_or_dir_toolchain(cfg, m)?;
 
@@ -929,7 +956,14 @@ fn doc(cfg: &Cfg, m: &ArgMatches) -> Result<()> {
         "index.html"
     };
 
-    Ok(cfg.open_docs_for_dir(&utils::current_dir()?, doc_url)?)
+    let cwd = &utils::current_dir()?;
+    if m.is_present("path") {
+        let doc_path = try!(cfg.doc_path_for_dir(cwd, doc_url));
+        println!("{}", doc_path.display());
+        Ok(())
+    } else {
+        Ok(cfg.open_docs_for_dir(cwd, doc_url)?)
+    }
 }
 
 fn man(cfg: &Cfg, m: &ArgMatches) -> Result<()> {

src/rustup-cli/rustup_mode.rs

@@ -13,7 +13,7 @@ repository = "https://github.com/rust-lang-nursery/rustup.rs"
 license = "MIT OR Apache-2.0"
 
 [dependencies]
-regex = "0.2.0"
+regex = "1.0.1"
 itertools = "0.7"
 ole32-sys = "0.2.0"
 url = "1.1.0"
@@ -23,9 +23,9 @@ xz2 = "0.1.3"
 walkdir = "2.0"
 toml = "0.4"
 sha2 = "0.7.0"
-remove_dir_all = "0.3"
+remove_dir_all = "0.5.1"
 rustup-utils = { path = "../rustup-utils" }
-error-chain = "0.11"
+error-chain = "0.12"
 
 [dev-dependencies]
 rustup-mock = { path = "../rustup-mock" }
@@ -40,4 +40,3 @@ libc = "0.2.0"
 
 [lib]
 name = "rustup_dist"
-

src/rustup-dist/Cargo.toml

@@ -24,4 +24,3 @@ wait-timeout = "0.1.3"
 [target."cfg(windows)".dependencies]
 winapi = "0.3"
 winreg = "0.5.0"
-

src/rustup-mock/Cargo.toml

@@ -13,17 +13,17 @@ license = "MIT OR Apache-2.0"
 
 [dependencies]
 download = { path = "../download" }
-error-chain = "0.11"
+error-chain = "0.12"
 libc = "0.2.0"
-rand = "0.4.2"
-remove_dir_all = "0.3.0"
+rand = "0.5.2"
+remove_dir_all = "0.5.1"
 scopeguard = "0.3.0"
 semver = "0.9.0"
 sha2 = "0.7.0"
 toml = "0.4"
 url = "1.1"
 
 [target."cfg(windows)".dependencies]
-winapi = { version = "0.3", features = ["combaseapi", "errhandlingapi", "fileapi", "handleapi", 
+winapi = { version = "0.3", features = ["combaseapi", "errhandlingapi", "fileapi", "handleapi",
     "ioapiset", "minwindef", "processthreadsapi", "shlobj", "shtypes", "userenv", "winbase", "winerror", "winnt", "winioctl"] }
 winreg = "0.5.0"

src/rustup-utils/Cargo.toml

@@ -240,7 +240,7 @@ impl TelemetryAnalysis {
         self.rustc_error_statistics.error_codes_with_counts = error_codes_with_counts;
         self.rustc_success_statistics = compute_rustc_percentiles(&rustc_successful_durations);
 
-        let error_list = error_list.into_iter().flatten();
+        let error_list = Itertools::flatten(error_list.into_iter());
 
         for e in error_list {
             let error_count = self.rustc_statistics

src/rustup/telemetry_analysis.rs

@@ -7,6 +7,7 @@ extern crate tempdir;
 
 use std::fs;
 use std::env::consts::EXE_SUFFIX;
+use std::path::MAIN_SEPARATOR;
 use std::process;
 use rustup_utils::raw;
 use rustup_mock::clitools::{self, expect_err, expect_ok, expect_ok_ex, expect_stderr_ok,
@@ -845,6 +846,34 @@ fn show_toolchain_env_not_installed() {
     });
 }
 
+#[test]
+fn show_active_toolchain() {
+    setup(&|config| {
+        expect_ok(config, &["rustup", "default", "nightly"]);
+        expect_ok_ex(
+            config,
+            &["rustup", "show", "active-toolchain"],
+            for_host!(
+                r"nightly-{0}
+"
+            ),
+            r"",
+        );
+    });
+}
+
+#[test]
+fn show_active_toolchain_none() {
+    setup(&|config| {
+        expect_ok_ex(
+            config,
+            &["rustup", "show", "active-toolchain"],
+            r"",
+            r"",
+        );
+    });
+}
+
 // #846
 #[test]
 fn set_default_host() {
@@ -1390,3 +1419,19 @@ fn file_override_with_target_info() {
         );
     });
 }
+
+#[test]
+fn docs_with_path() {
+    setup(&|config| {
+        expect_ok(config, &["rustup", "default", "stable"]);
+
+        let mut cmd = clitools::cmd(config, "rustup", &["doc", "--path"]);
+        clitools::env(config, &mut cmd);
+        let out = cmd.output().unwrap();
+
+        let stdout = String::from_utf8(out.stdout).unwrap();
+        let path = format!("share{}doc{}rust{}html",
+            MAIN_SEPARATOR, MAIN_SEPARATOR, MAIN_SEPARATOR);
+        assert!(stdout.contains(path.as_str()));
+    });
+}