From 4820b9ec84da6e024602cc67259bb444108dce4f Mon Sep 17 00:00:00 2001 From: zohnannor Date: Mon, 22 Aug 2022 01:39:04 +0300 Subject: [PATCH 1/6] Bump clap to 3.0 --- .gitignore | 1 + Cargo.lock | 87 ++-- Cargo.toml | 3 +- src/cli/rustup_mode.rs | 867 ++++++++++++++++++++-------------------- src/cli/setup_mode.rs | 110 ++--- tests/suite/cli_misc.rs | 8 +- 6 files changed, 555 insertions(+), 521 deletions(-) diff --git a/.gitignore b/.gitignore index eb4221f2ac..ca4609753d 100644 --- a/.gitignore +++ b/.gitignore @@ -3,6 +3,7 @@ /Cargo.lock /.settings /.idea/ +/.vscode/ *~ /**/target /home diff --git a/Cargo.lock b/Cargo.lock index ddfd5cfb0d..dd0c67ea47 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -66,15 +66,6 @@ dependencies = [ "libc", ] -[[package]] -name = "ansi_term" -version = "0.12.1" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "d52a9bb7ec0cf484c551830a7ce27bd20d67eac647e1befb56b0be4ee39a55d2" -dependencies = [ - "winapi", -] - [[package]] name = "anyhow" version = "1.0.69" @@ -317,18 +308,36 @@ dependencies = [ [[package]] name = "clap" -version = "2.34.0" +version = "3.2.23" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "a0610544180c38b88101fecf2dd634b174a62eef6946f84dfc6a7127512b381c" +checksum = "71655c45cb9845d3270c9d6df84ebe72b4dad3c2ba3f7023ad47c144e4e473a5" dependencies = [ - "ansi_term", "atty", "bitflags", - "strsim 0.8.0", - "term_size", + "clap_lex", + "indexmap", + "strsim", + "termcolor", + "terminal_size", "textwrap", - "unicode-width", - "vec_map", +] + +[[package]] +name = "clap_complete" +version = "3.2.5" +source = "registry+https://github.com/rust-lang/crates.io-index" +checksum = "3f7a2e0a962c45ce25afce14220bc24f9dade0a1787f185cecf96bfba7847cd8" +dependencies = [ + "clap", +] + +[[package]] +name = "clap_lex" +version = "0.2.4" +source = "registry+https://github.com/rust-lang/crates.io-index" +checksum = "2850f2f5a82cbf437dd5af4d49848fbdfc27c157c3d010345776f952765261c5" +dependencies = [ + "os_str_bytes", ] [[package]] @@ -1715,6 +1724,12 @@ dependencies = [ "windows-sys 0.45.0", ] +[[package]] +name = "os_str_bytes" +version = "6.4.1" +source = "registry+https://github.com/rust-lang/crates.io-index" +checksum = "9b7820b9daea5457c9f21c69448905d723fbd21136ccf521748f23fd49e723ee" + [[package]] name = "p256" version = "0.8.1" @@ -2177,6 +2192,7 @@ dependencies = [ "cfg-if 1.0.0", "chrono", "clap", + "clap_complete", "download", "effective-limits", "enum-map", @@ -2202,7 +2218,7 @@ dependencies = [ "serde", "sha2 0.10.6", "sharded-slab", - "strsim 0.10.0", + "strsim", "tar", "tempfile", "term", @@ -2578,12 +2594,6 @@ dependencies = [ "precomputed-hash", ] -[[package]] -name = "strsim" -version = "0.8.0" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "8ea5119cdb4c55b55d432abb513a0429384878c15dde60cc77b1c99de1a95a6a" - [[package]] name = "strsim" version = "0.10.0" @@ -2671,32 +2681,31 @@ dependencies = [ ] [[package]] -name = "term_size" -version = "0.3.2" +name = "termcolor" +version = "1.2.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "1e4129646ca0ed8f45d09b929036bafad5377103edd06e50bf574b353d2b08d9" +checksum = "be55cf8942feac5c765c2c993422806843c9a9a45d4d5c407ad6dd2ea95eb9b6" dependencies = [ - "libc", - "winapi", + "winapi-util", ] [[package]] -name = "termcolor" -version = "1.2.0" +name = "terminal_size" +version = "0.2.5" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "be55cf8942feac5c765c2c993422806843c9a9a45d4d5c407ad6dd2ea95eb9b6" +checksum = "4c9afddd2cec1c0909f06b00ef33f94ab2cc0578c4a610aa208ddfec8aa2b43a" dependencies = [ - "winapi-util", + "rustix", + "windows-sys 0.45.0", ] [[package]] name = "textwrap" -version = "0.11.0" +version = "0.16.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "d326610f408c7a4eb6f51c37c330e496b08506c9457c9d34287ecc38809fb060" +checksum = "222a222a5bfe1bba4a77b45ec488a741b3cb8872e5e499451fd7d0129c9c7c3d" dependencies = [ - "term_size", - "unicode-width", + "terminal_size", ] [[package]] @@ -3015,12 +3024,6 @@ version = "0.2.15" source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "accd4ea62f7bb7a82fe23066fb0957d48ef677f6eeb8215f372f52e48bb32426" -[[package]] -name = "vec_map" -version = "0.8.2" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "f1bddf1187be692e79c5ffeab891132dfb0f236ed36a43c7ed39f1165ee20191" - [[package]] name = "version_check" version = "0.9.4" diff --git a/Cargo.toml b/Cargo.toml index 80de6608fa..f4c413347b 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -29,7 +29,8 @@ no-self-update = [] anyhow.workspace = true cfg-if = "1.0" chrono = "0.4" -clap = {version = "2", features = ["wrap_help"]} +clap = {version = "3", features = ["wrap_help"]} +clap_complete = "3" download = {path = "download", default-features = false} effective-limits = "0.5.5" enum-map = "2.4.2" diff --git a/src/cli/rustup_mode.rs b/src/cli/rustup_mode.rs index a8b36567ae..c856f11833 100644 --- a/src/cli/rustup_mode.rs +++ b/src/cli/rustup_mode.rs @@ -1,11 +1,15 @@ use std::fmt; use std::io::Write; use std::path::{Path, PathBuf}; -use std::process::Command; +use std::process; use std::str::FromStr; use anyhow::{anyhow, bail, Error, Result}; -use clap::{App, AppSettings, Arg, ArgGroup, ArgMatches, Shell, SubCommand}; +use clap::{ + builder::{EnumValueParser, PossibleValuesParser}, + AppSettings, Arg, ArgAction, ArgEnum, ArgGroup, ArgMatches, Command, PossibleValue, +}; +use clap_complete::Shell; use super::help::*; use super::self_update; @@ -64,22 +68,14 @@ pub fn main() -> Result { self_update::cleanup_self_updater()?; use clap::ErrorKind::*; - let matches = match cli().get_matches_from_safe(process().args_os()) { + let matches = match cli().try_get_matches_from(process().args_os()) { Ok(matches) => Ok(matches), - Err(clap::Error { - kind: HelpDisplayed, - message, - .. - }) => { - writeln!(process().stdout().lock(), "{message}")?; + Err(err) if err.kind() == DisplayHelp => { + writeln!(process().stdout().lock(), "{}", err)?; return Ok(utils::ExitCode(0)); } - Err(clap::Error { - kind: VersionDisplayed, - message, - .. - }) => { - writeln!(process().stdout().lock(), "{message}")?; + Err(err) if err.kind() == DisplayVersion => { + writeln!(process().stdout().lock(), "{}", err)?; info!("This is the version for the rustup toolchain manager, not the rustc compiler."); fn rustc_version() -> std::result::Result> { @@ -103,28 +99,25 @@ pub fn main() -> Result { return Ok(utils::ExitCode(0)); } - Err(e) => { + Err(err) => { + if [ + InvalidSubcommand, + UnknownArgument, + DisplayHelpOnMissingArgumentOrSubcommand, + ] + .contains(&err.kind()) { - let clap::Error { kind, message, .. } = &e; - if [ - InvalidSubcommand, - UnknownArgument, - MissingArgumentOrSubcommand, - ] - .contains(kind) - { - writeln!(process().stdout().lock(), "{message}")?; - return Ok(utils::ExitCode(1)); - } + writeln!(process().stdout().lock(), "{}", err)?; + return Ok(utils::ExitCode(1)); } - Err(e) + Err(err) } }?; - let verbose = matches.is_present("verbose"); - let quiet = matches.is_present("quiet"); + let verbose = matches.get_flag("verbose"); + let quiet = matches.get_flag("quiet"); let cfg = &mut common::set_globals(verbose, quiet)?; - if let Some(t) = matches.value_of("+toolchain") { + if let Some(t) = matches.get_one::("+toolchain") { cfg.set_toolchain_override(&t[1..]); } @@ -135,120 +128,128 @@ pub fn main() -> Result { cfg.check_metadata_version()?; Ok(match matches.subcommand() { - ("dump-testament", _) => common::dump_testament()?, - ("show", Some(c)) => match c.subcommand() { - ("active-toolchain", Some(m)) => handle_epipe(show_active_toolchain(cfg, m))?, - ("home", Some(_)) => handle_epipe(show_rustup_home(cfg))?, - ("profile", Some(_)) => handle_epipe(show_profile(cfg))?, - ("keys", Some(_)) => handle_epipe(show_keys(cfg))?, - (_, _) => handle_epipe(show(cfg, c))?, + Some(("dump-testament", _)) => common::dump_testament()?, + Some(("show", c)) => match c.subcommand() { + Some(("active-toolchain", m)) => handle_epipe(show_active_toolchain(cfg, m))?, + Some(("home", _)) => handle_epipe(show_rustup_home(cfg))?, + Some(("profile", _)) => handle_epipe(show_profile(cfg))?, + Some(("keys", _)) => handle_epipe(show_keys(cfg))?, + _ => handle_epipe(show(cfg, c))?, }, - ("install", Some(m)) => deprecated("toolchain install", cfg, m, update)?, - ("update", Some(m)) => update(cfg, m)?, - ("check", Some(_)) => check_updates(cfg)?, - ("uninstall", Some(m)) => deprecated("toolchain uninstall", cfg, m, toolchain_remove)?, - ("default", Some(m)) => default_(cfg, m)?, - ("toolchain", Some(c)) => match c.subcommand() { - ("install", Some(m)) => update(cfg, m)?, - ("list", Some(m)) => handle_epipe(toolchain_list(cfg, m))?, - ("link", Some(m)) => toolchain_link(cfg, m)?, - ("uninstall", Some(m)) => toolchain_remove(cfg, m)?, - (_, _) => unreachable!(), + Some(("install", m)) => deprecated("toolchain install", cfg, m, update)?, + Some(("update", m)) => update(cfg, m)?, + Some(("check", _)) => check_updates(cfg)?, + Some(("uninstall", m)) => deprecated("toolchain uninstall", cfg, m, toolchain_remove)?, + Some(("default", m)) => default_(cfg, m)?, + Some(("toolchain", c)) => match c.subcommand() { + Some(("install", m)) => update(cfg, m)?, + Some(("list", m)) => handle_epipe(toolchain_list(cfg, m))?, + Some(("link", m)) => toolchain_link(cfg, m)?, + Some(("uninstall", m)) => toolchain_remove(cfg, m)?, + _ => unreachable!(), }, - ("target", Some(c)) => match c.subcommand() { - ("list", Some(m)) => handle_epipe(target_list(cfg, m))?, - ("add", Some(m)) => target_add(cfg, m)?, - ("remove", Some(m)) => target_remove(cfg, m)?, - (_, _) => unreachable!(), + Some(("target", c)) => match c.subcommand() { + Some(("list", m)) => handle_epipe(target_list(cfg, m))?, + Some(("add", m)) => target_add(cfg, m)?, + Some(("remove", m)) => target_remove(cfg, m)?, + _ => unreachable!(), }, - ("component", Some(c)) => match c.subcommand() { - ("list", Some(m)) => handle_epipe(component_list(cfg, m))?, - ("add", Some(m)) => component_add(cfg, m)?, - ("remove", Some(m)) => component_remove(cfg, m)?, - (_, _) => unreachable!(), + Some(("component", c)) => match c.subcommand() { + Some(("list", m)) => handle_epipe(component_list(cfg, m))?, + Some(("add", m)) => component_add(cfg, m)?, + Some(("remove", m)) => component_remove(cfg, m)?, + _ => unreachable!(), }, - ("override", Some(c)) => match c.subcommand() { - ("list", Some(_)) => handle_epipe(common::list_overrides(cfg))?, - ("set", Some(m)) => override_add(cfg, m)?, - ("unset", Some(m)) => override_remove(cfg, m)?, - (_, _) => unreachable!(), + Some(("override", c)) => match c.subcommand() { + Some(("list", _)) => handle_epipe(common::list_overrides(cfg))?, + Some(("set", m)) => override_add(cfg, m)?, + Some(("unset", m)) => override_remove(cfg, m)?, + _ => unreachable!(), }, - ("run", Some(m)) => run(cfg, m)?, - ("which", Some(m)) => which(cfg, m)?, - ("doc", Some(m)) => doc(cfg, m)?, - ("man", Some(m)) => man(cfg, m)?, - ("self", Some(c)) => match c.subcommand() { - ("update", Some(_)) => self_update::update(cfg)?, - ("uninstall", Some(m)) => self_uninstall(m)?, - (_, _) => unreachable!(), + Some(("run", m)) => run(cfg, m)?, + Some(("which", m)) => which(cfg, m)?, + Some(("doc", m)) => doc(cfg, m)?, + Some(("man", m)) => man(cfg, m)?, + Some(("self", c)) => match c.subcommand() { + Some(("update", _)) => self_update::update(cfg)?, + Some(("uninstall", m)) => self_uninstall(m)?, + _ => unreachable!(), }, - ("set", Some(c)) => match c.subcommand() { - ("default-host", Some(m)) => set_default_host_triple(cfg, m)?, - ("profile", Some(m)) => set_profile(cfg, m)?, - ("auto-self-update", Some(m)) => set_auto_self_update(cfg, m)?, - (_, _) => unreachable!(), + Some(("set", c)) => match c.subcommand() { + Some(("default-host", m)) => set_default_host_triple(cfg, m)?, + Some(("profile", m)) => set_profile(cfg, m)?, + Some(("auto-self-update", m)) => set_auto_self_update(cfg, m)?, + _ => unreachable!(), }, - ("completions", Some(c)) => { - if let Some(shell) = c.value_of("shell") { + Some(("completions", c)) => { + if let Some(&shell) = c.get_one::("shell") { (output_completion_script( - shell.parse::().unwrap(), - c.value_of("command") - .and_then(|cmd| cmd.parse::().ok()) + shell, + c.get_one::("command") + .copied() .unwrap_or(CompletionCommand::Rustup), ))? } else { unreachable!() } } - (_, _) => unreachable!(), + _ => unreachable!(), }) } -pub(crate) fn cli() -> App<'static, 'static> { - let mut app = App::new("rustup") +pub(crate) fn cli() -> Command<'static> { + let mut app = Command::new("rustup") .version(common::version()) .about("The Rust toolchain installer") .after_help(RUSTUP_HELP) - .setting(AppSettings::VersionlessSubcommands) - .setting(AppSettings::DeriveDisplayOrder) - .setting(AppSettings::SubcommandRequiredElseHelp) + .global_setting(AppSettings::DeriveDisplayOrder) + .arg_required_else_help(true) .arg( verbose_arg("Enable verbose output"), ) + .help_template( + "\ +{name} {version} +{about} + +USAGE: + {usage} + +{all-args}", + ) .arg( - Arg::with_name("quiet") + Arg::new("quiet") .conflicts_with("verbose") .help("Disable progress output") - .short("q") - .long("quiet"), + .short('q') + .long("quiet") + .action(ArgAction::SetTrue), ) .arg( - Arg::with_name("+toolchain") + Arg::new("+toolchain") .help("release channel (e.g. +stable) or custom toolchain to set override") - .validator(|s| { + .value_parser(|s: &str| { if s.starts_with('+') { - Ok(()) + Ok(s.to_owned()) } else { - Err("Toolchain overrides must begin with '+'".into()) + Err("Toolchain overrides must begin with '+'".to_owned()) } }), ) .subcommand( - SubCommand::with_name("dump-testament") + Command::new("dump-testament") .about("Dump information about the build") - .setting(AppSettings::Hidden), // Not for users, only CI + .hide(true), // Not for users, only CI ) .subcommand( - SubCommand::with_name("show") + Command::new("show") .about("Show the active and installed toolchains or profiles") .after_help(SHOW_HELP) .arg( verbose_arg("Enable verbose output with rustc information for all installed toolchains"), ) - .setting(AppSettings::VersionlessSubcommands) - .setting(AppSettings::DeriveDisplayOrder) .subcommand( - SubCommand::with_name("active-toolchain") + Command::new("active-toolchain") .about("Show the active toolchain") .after_help(SHOW_ACTIVE_TOOLCHAIN_HELP) .arg( @@ -256,245 +257,265 @@ pub(crate) fn cli() -> App<'static, 'static> { ), ) .subcommand( - SubCommand::with_name("home") + Command::new("home") .about("Display the computed value of RUSTUP_HOME"), ) - .subcommand(SubCommand::with_name("profile").about("Show the current profile")) - .subcommand(SubCommand::with_name("keys").about("Display the known PGP keys")), + .subcommand(Command::new("profile").about("Show the current profile")) + .subcommand(Command::new("keys").about("Display the known PGP keys")), ) .subcommand( - SubCommand::with_name("install") + Command::new("install") .about("Update Rust toolchains") .after_help(INSTALL_HELP) - .setting(AppSettings::Hidden) // synonym for 'toolchain install' + .hide(true) // synonym for 'toolchain install' .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .required(true) - .multiple(true), + .takes_value(true) + .multiple_values(true) ) .arg( - Arg::with_name("profile") + Arg::new("profile") .long("profile") - .takes_value(true) - .possible_values(Profile::names()) - .required(false), + .value_parser(PossibleValuesParser::new(Profile::names())) + .takes_value(true), ) .arg( - Arg::with_name("no-self-update") + Arg::new("no-self-update") .help("Don't perform self-update when running the `rustup install` command") .long("no-self-update") - .takes_value(false), + .action(ArgAction::SetTrue) ) .arg( - Arg::with_name("force") + Arg::new("force") .help("Force an update, even if some components are missing") .long("force") - .takes_value(false), + .action(ArgAction::SetTrue) ).arg( - Arg::with_name("force-non-host") + Arg::new("force-non-host") .help("Install toolchains that require an emulator. See https://github.com/rust-lang/rustup/wiki/Non-host-toolchains") .long("force-non-host") - .takes_value(false) + .action(ArgAction::SetTrue) ), ) .subcommand( - SubCommand::with_name("uninstall") + Command::new("uninstall") .about("Uninstall Rust toolchains") - .setting(AppSettings::Hidden) // synonym for 'toolchain uninstall' + .hide(true) // synonym for 'toolchain uninstall' .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .required(true) - .multiple(true), + .takes_value(true) + .multiple_values(true), ), ) .subcommand( - SubCommand::with_name("update") + Command::new("update") .about("Update Rust toolchains and rustup") .aliases(&["upgrade", "up"]) .after_help(UPDATE_HELP) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .required(false) - .multiple(true), + .takes_value(true) + .multiple_values(true), ) .arg( - Arg::with_name("no-self-update") + Arg::new("no-self-update") .help("Don't perform self update when running the `rustup update` command") .long("no-self-update") - .takes_value(false), + .action(ArgAction::SetTrue) ) .arg( - Arg::with_name("force") + Arg::new("force") .help("Force an update, even if some components are missing") .long("force") - .takes_value(false), + .action(ArgAction::SetTrue) ) .arg( - Arg::with_name("force-non-host") + Arg::new("force-non-host") .help("Install toolchains that require an emulator. See https://github.com/rust-lang/rustup/wiki/Non-host-toolchains") .long("force-non-host") - .takes_value(false)), + .action(ArgAction::SetTrue) + ), ) - .subcommand(SubCommand::with_name("check").about("Check for updates to Rust toolchains and rustup")) + .subcommand(Command::new("check").about("Check for updates to Rust toolchains and rustup")) .subcommand( - SubCommand::with_name("default") + Command::new("default") .about("Set the default toolchain") .after_help(DEFAULT_HELP) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) - .required(false), + .required(false) + .takes_value(true), ), ) .subcommand( - SubCommand::with_name("toolchain") + Command::new("toolchain") .about("Modify or query the installed toolchains") .after_help(TOOLCHAIN_HELP) - .setting(AppSettings::VersionlessSubcommands) - .setting(AppSettings::DeriveDisplayOrder) - .setting(AppSettings::SubcommandRequiredElseHelp) + .subcommand_required(true) + .arg_required_else_help(true) .subcommand( - SubCommand::with_name("list") + Command::new("list") .about("List installed toolchains") .arg( verbose_arg("Enable verbose output with toolchain information"), ), ) .subcommand( - SubCommand::with_name("install") + Command::new("install") .about("Install or update a given toolchain") .aliases(&["update", "add"]) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .required(true) - .multiple(true), - ) - .arg( - Arg::with_name("profile") - .long("profile") .takes_value(true) - .possible_values(Profile::names()) - .required(false), + .multiple_values(true), ) .arg( - Arg::with_name("no-self-update") - .help( - "Don't perform self update when running the\ - `rustup toolchain install` command", - ) - .long("no-self-update") - .takes_value(false), + Arg::new("profile") + .long("profile") + .value_parser(PossibleValuesParser::new(Profile::names())) + .takes_value(true), ) .arg( - Arg::with_name("components") + Arg::new("components") .help("Add specific components on installation") .long("component") - .short("c") + .short('c') .takes_value(true) - .multiple(true) - .use_delimiter(true), + .multiple_values(true) + .use_value_delimiter(true) + .action(ArgAction::Append), ) .arg( - Arg::with_name("targets") + Arg::new("targets") .help("Add specific targets on installation") .long("target") - .short("t") + .short('t') + .takes_value(true) + .multiple_values(true) + .multiple_occurrences(true) + .use_value_delimiter(true), + ) + .arg( + Arg::new("no-self-update") + .help( + "Don't perform self update when running the\ + `rustup toolchain install` command", + ) + .long("no-self-update") .takes_value(true) - .multiple(true) - .use_delimiter(true), + .action(ArgAction::SetTrue) ) .arg( - Arg::with_name("force") + Arg::new("force") .help("Force an update, even if some components are missing") .long("force") - .takes_value(false), + .action(ArgAction::SetTrue) ) .arg( - Arg::with_name("allow-downgrade") + Arg::new("allow-downgrade") .help("Allow rustup to downgrade the toolchain to satisfy your component choice") .long("allow-downgrade") - .takes_value(false), + .action(ArgAction::SetTrue) ) .arg( - Arg::with_name("force-non-host") + Arg::new("force-non-host") .help("Install toolchains that require an emulator. See https://github.com/rust-lang/rustup/wiki/Non-host-toolchains") .long("force-non-host") - .takes_value(false)), + .action(ArgAction::SetTrue) + ), ) .subcommand( - SubCommand::with_name("uninstall") + Command::new("uninstall") .about("Uninstall a toolchain") .alias("remove") .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .required(true) - .multiple(true), + .takes_value(true) + .multiple_values(true), ), ) .subcommand( - SubCommand::with_name("link") + Command::new("link") .about("Create a custom toolchain by symlinking to a directory") .after_help(TOOLCHAIN_LINK_HELP) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help("Custom toolchain name") .required(true), ) .arg( - Arg::with_name("path") + Arg::new("path") .help("Path to the directory") .required(true), ), ), ) .subcommand( - SubCommand::with_name("target") + Command::new("target") .about("Modify a toolchain's supported targets") - .setting(AppSettings::VersionlessSubcommands) - .setting(AppSettings::DeriveDisplayOrder) - .setting(AppSettings::SubcommandRequiredElseHelp) + .subcommand_required(true) + .arg_required_else_help(true) .subcommand( - SubCommand::with_name("list") + Command::new("list") .about("List installed and available targets") .arg( - Arg::with_name("installed") - .long("--installed") - .help("List only installed targets"), - ) - .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), + ) + .arg( + Arg::new("installed") + .long("installed") + .help("List only installed targets") + .action(ArgAction::SetTrue) ), ) .subcommand( - SubCommand::with_name("add") + Command::new("add") .about("Add a target to a Rust toolchain") .alias("install") - .arg(Arg::with_name("target").required(true).multiple(true).help( - "List of targets to install; \ - \"all\" installs all available targets", - )) .arg( - Arg::with_name("toolchain") + Arg::new("target") + .required(true) + .takes_value(true) + .multiple_values(true) + .help( + "List of targets to install; \ + \"all\" installs all available targets" + ) + ) + .arg( + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), ), ) .subcommand( - SubCommand::with_name("remove") + Command::new("remove") .about("Remove a target from a Rust toolchain") .alias("uninstall") - .arg(Arg::with_name("target").required(true).multiple(true)) .arg( - Arg::with_name("toolchain") + Arg::new("target") + .help("List of targets to uninstall") + .required(true) + .takes_value(true) + .multiple_values(true) + ) + .arg( + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), @@ -502,169 +523,183 @@ pub(crate) fn cli() -> App<'static, 'static> { ), ) .subcommand( - SubCommand::with_name("component") + Command::new("component") .about("Modify a toolchain's installed components") - .setting(AppSettings::VersionlessSubcommands) - .setting(AppSettings::DeriveDisplayOrder) - .setting(AppSettings::SubcommandRequiredElseHelp) + .subcommand_required(true) + .arg_required_else_help(true) .subcommand( - SubCommand::with_name("list") + Command::new("list") .about("List installed and available components") .arg( - Arg::with_name("installed") - .long("--installed") - .help("List only installed components"), - ) - .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), + ) + .arg( + Arg::new("installed") + .long("installed") + .help("List only installed components") + .action(ArgAction::SetTrue) ), ) .subcommand( - SubCommand::with_name("add") + Command::new("add") .about("Add a component to a Rust toolchain") - .arg(Arg::with_name("component").required(true).multiple(true)) + .arg(Arg::new("component").required(true) + .takes_value(true).multiple_values(true)) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), ) - .arg(Arg::with_name("target").long("target").takes_value(true)), + .arg( + Arg::new("target") + .long("target") + .takes_value(true) + ), ) .subcommand( - SubCommand::with_name("remove") + Command::new("remove") .about("Remove a component from a Rust toolchain") - .arg(Arg::with_name("component").required(true).multiple(true)) + .arg(Arg::new("component").required(true) + .takes_value(true).multiple_values(true)) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), ) - .arg(Arg::with_name("target").long("target").takes_value(true)), + .arg( + Arg::new("target") + .long("target") + .takes_value(true) + ), ), ) .subcommand( - SubCommand::with_name("override") + Command::new("override") .about("Modify directory toolchain overrides") .after_help(OVERRIDE_HELP) - .setting(AppSettings::VersionlessSubcommands) - .setting(AppSettings::DeriveDisplayOrder) - .setting(AppSettings::SubcommandRequiredElseHelp) + .subcommand_required(true) + .arg_required_else_help(true) .subcommand( - SubCommand::with_name("list").about("List directory toolchain overrides"), + Command::new("list").about("List directory toolchain overrides"), ) .subcommand( - SubCommand::with_name("set") + Command::new("set") .about("Set the override toolchain for a directory") .alias("add") .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) - .required(true), + .required(true) + .takes_value(true), ) .arg( - Arg::with_name("path") + Arg::new("path") .long("path") .takes_value(true) .help("Path to the directory"), ), ) .subcommand( - SubCommand::with_name("unset") + Command::new("unset") .about("Remove the override toolchain for a directory") .after_help(OVERRIDE_UNSET_HELP) .alias("remove") .arg( - Arg::with_name("path") + Arg::new("path") .long("path") .takes_value(true) .help("Path to the directory"), ) .arg( - Arg::with_name("nonexistent") + Arg::new("nonexistent") .long("nonexistent") - .takes_value(false) - .help("Remove override toolchain for all nonexistent directories"), + .help("Remove override toolchain for all nonexistent directories") + .action(ArgAction::SetTrue), ), ), ) .subcommand( - SubCommand::with_name("run") + Command::new("run") .about("Run a command with an environment configured for a given toolchain") .after_help(RUN_HELP) - .setting(AppSettings::TrailingVarArg) - .arg( - Arg::with_name("install") - .help("Install the requested toolchain if needed") - .long("install"), - ) + .trailing_var_arg(true) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) - .required(true), + .required(true) + .takes_value(true), ) .arg( - Arg::with_name("command") + Arg::new("command") .required(true) - .multiple(true) - .use_delimiter(false), + .takes_value(true) + .multiple_values(true) + .use_value_delimiter(false), + ) + .arg( + Arg::new("install") + .help("Install the requested toolchain if needed") + .long("install") + .action(ArgAction::SetTrue), ), ) .subcommand( - SubCommand::with_name("which") + Command::new("which") .about("Display which binary will be run for a given command") - .arg(Arg::with_name("command").required(true)) + .arg(Arg::new("command").required(true)) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), ), ) .subcommand( - SubCommand::with_name("doc") + Command::new("doc") .alias("docs") .about("Open the documentation for the current toolchain") .after_help(DOC_HELP) .arg( - Arg::with_name("path") + Arg::new("path") .long("path") - .help("Only print the path to the documentation"), - ) - .args( - &DOCS_DATA - .iter() - .map(|(name, help_msg, _)| Arg::with_name(name).long(name).help(help_msg)) - .collect::>(), + .help("Only print the path to the documentation") + .action(ArgAction::SetTrue), ) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), ) + .arg(Arg::new("topic").help(TOPIC_ARG_HELP)) .group( - ArgGroup::with_name("page").args( + ArgGroup::new("page").args( &DOCS_DATA .iter() .map(|(name, _, _)| *name) .collect::>(), ), ) - .arg(Arg::with_name("topic").help(TOPIC_ARG_HELP)), + .args( + &DOCS_DATA + .iter() + .map(|&(name, help_msg, _)| Arg::new(name).long(name).help(help_msg).action(ArgAction::SetTrue)) + .collect::>(), + ), ); if cfg!(not(target_os = "windows")) { app = app.subcommand( - SubCommand::with_name("man") + Command::new("man") .about("View the man page for a given command") - .arg(Arg::with_name("command").required(true)) + .arg(Arg::new("command").required(true)) .arg( - Arg::with_name("toolchain") + Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .long("toolchain") .takes_value(true), @@ -674,91 +709,78 @@ pub(crate) fn cli() -> App<'static, 'static> { app = app .subcommand( - SubCommand::with_name("self") + Command::new("self") .about("Modify the rustup installation") - .setting(AppSettings::VersionlessSubcommands) - .setting(AppSettings::DeriveDisplayOrder) - .setting(AppSettings::SubcommandRequiredElseHelp) - .subcommand( - SubCommand::with_name("update").about("Download and install updates to rustup"), - ) + .subcommand_required(true) + .arg_required_else_help(true) + .subcommand(Command::new("update").about("Download and install updates to rustup")) .subcommand( - SubCommand::with_name("uninstall") + Command::new("uninstall") .about("Uninstall rustup.") - .arg(Arg::with_name("no-prompt").short("y")), + .arg(Arg::new("no-prompt").short('y').action(ArgAction::SetTrue)), ) .subcommand( - SubCommand::with_name("upgrade-data") - .about("Upgrade the internal data format."), + Command::new("upgrade-data").about("Upgrade the internal data format."), ), ) .subcommand( - SubCommand::with_name("set") + Command::new("set") .about("Alter rustup settings") - .setting(AppSettings::SubcommandRequiredElseHelp) + .subcommand_required(true) + .arg_required_else_help(true) .subcommand( - SubCommand::with_name("default-host") + Command::new("default-host") .about("The triple used to identify toolchains when not specified") - .arg(Arg::with_name("host_triple").required(true)), + .arg(Arg::new("host_triple").required(true)), ) .subcommand( - SubCommand::with_name("profile") + Command::new("profile") .about("The default components installed") .arg( - Arg::with_name("profile-name") + Arg::new("profile-name") .required(true) - .possible_values(Profile::names()) + .value_parser(PossibleValuesParser::new(Profile::names())) .default_value(Profile::default_name()), ), ) .subcommand( - SubCommand::with_name("auto-self-update") + Command::new("auto-self-update") .about("The rustup auto self update mode") .arg( - Arg::with_name("auto-self-update-mode") + Arg::new("auto-self-update-mode") .required(true) - .possible_values(SelfUpdateMode::modes()) + .value_parser(PossibleValuesParser::new(SelfUpdateMode::modes())) .default_value(SelfUpdateMode::default_mode()), ), ), ); - // Clap provides no good way to say that help should be printed in all - // cases where an argument without a default is not provided. The following - // creates lists out all the conditions where the "shell" argument are - // provided and give the default of "rustup". This way if "shell" is not - // provided then the help will still be printed. - let completion_defaults = Shell::variants() - .iter() - .map(|&shell| ("shell", Some(shell), "rustup")) - .collect::>(); - app.subcommand( - SubCommand::with_name("completions") + Command::new("completions") .about("Generate tab-completion scripts for your shell") .after_help(COMPLETIONS_HELP) - .setting(AppSettings::ArgRequiredElseHelp) - .arg(Arg::with_name("shell").possible_values(&Shell::variants())) + .arg_required_else_help(true) + .arg(Arg::new("shell").value_parser(EnumValueParser::::new())) .arg( - Arg::with_name("command") - .possible_values(&CompletionCommand::variants()) - .default_value_ifs(&completion_defaults[..]), + Arg::new("command") + .value_parser(EnumValueParser::::new()) + .default_missing_value("rustup"), ), ) } -fn verbose_arg(help: &str) -> Arg<'_, '_> { - Arg::with_name("verbose") +fn verbose_arg(help: &str) -> Arg<'_> { + Arg::new("verbose") .help(help) - .takes_value(false) - .short("v") + .short('v') .long("verbose") + .action(ArgAction::SetTrue) } -fn maybe_upgrade_data(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { +fn maybe_upgrade_data(cfg: &Cfg, m: &ArgMatches) -> Result { match m.subcommand() { - ("self", Some(c)) => match c.subcommand() { - ("upgrade-data", Some(_)) => { + Some(("self", c)) => match c.subcommand() { + Some(("upgrade-data", _)) => { cfg.upgrade_data()?; Ok(true) } @@ -851,9 +873,8 @@ fn default_bare_triple_check(cfg: &Cfg, name: &str) -> Result<()> { Ok(()) } -fn default_(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - if m.is_present("toolchain") { - let toolchain = m.value_of("toolchain").unwrap(); +fn default_(cfg: &Cfg, m: &ArgMatches) -> Result { + if let Some(toolchain) = m.get_one::("toolchain") { default_bare_triple_check(cfg, toolchain)?; let toolchain = cfg.get_toolchain(toolchain, false)?; @@ -937,7 +958,7 @@ fn check_updates(cfg: &Cfg) -> Result { Ok(utils::ExitCode(0)) } -fn update(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { +fn update(cfg: &mut Cfg, m: &ArgMatches) -> Result { let self_update_mode = cfg.get_self_update_mode()?; // Priority: no-self-update feature > self_update_mode > no-self-update args. // Update only if rustup does **not** have the no-self-update feature, @@ -945,9 +966,9 @@ fn update(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { // and has **no** no-self-update parameter. let self_update = !self_update::NEVER_SELF_UPDATE && self_update_mode == SelfUpdateMode::Enable - && !m.is_present("no-self-update"); - let forced = m.is_present("force-non-host"); - if let Some(p) = m.value_of("profile") { + && !m.get_flag("no-self-update"); + let forced = m.get_flag("force-non-host"); + if let Ok(Some(p)) = m.try_get_one::("profile") { let p = Profile::from_str(p)?; cfg.set_profile_override(p); } @@ -955,7 +976,7 @@ fn update(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { if cfg.get_profile()? == Profile::Complete { warn!("{}", common::WARN_COMPLETE_PROFILE); } - if let Some(names) = m.values_of("toolchain") { + if let Ok(Some(names)) = m.try_get_many::("toolchain") { for name in names { update_bare_triple_check(cfg, name)?; @@ -986,17 +1007,19 @@ fn update(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { let status = if !toolchain.is_custom() { let components: Vec<_> = m - .values_of("components") - .map(|v| v.collect()) - .unwrap_or_else(Vec::new); + .try_get_many::("components") + .ok() + .flatten() + .map_or_else(Vec::new, |v| v.map(|s| &**s).collect()); let targets: Vec<_> = m - .values_of("targets") - .map(|v| v.collect()) - .unwrap_or_else(Vec::new); + .try_get_many::("targets") + .ok() + .flatten() + .map_or_else(Vec::new, |v| v.map(|s| &**s).collect()); let distributable = DistributableToolchain::new(&toolchain)?; Some(distributable.install_from_dist( - m.is_present("force"), - m.is_present("allow-downgrade"), + m.get_flag("force"), + matches!(m.try_get_one::("allow-downgrade"), Ok(Some(true))), &components, &targets, None, @@ -1025,7 +1048,7 @@ fn update(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { common::self_update(|| Ok(utils::ExitCode(0)))?; } } else { - common::update_all_channels(cfg, self_update, m.is_present("force"))?; + common::update_all_channels(cfg, self_update, m.get_flag("force"))?; info!("cleaning up downloads & tmp directories"); utils::delete_dir_contents(&cfg.download_dir); cfg.temp_cfg.clean(); @@ -1043,20 +1066,19 @@ fn update(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn run(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - let toolchain = m.value_of("toolchain").unwrap(); - let args = m.values_of("command").unwrap(); +fn run(cfg: &Cfg, m: &ArgMatches) -> Result { + let toolchain = m.get_one::("toolchain").unwrap(); + let args = m.get_many::("command").unwrap(); let args: Vec<_> = args.collect(); - let cmd = cfg.create_command_for_toolchain(toolchain, m.is_present("install"), args[0])?; + let cmd = cfg.create_command_for_toolchain(toolchain, m.get_flag("install"), args[0])?; let code = command::run_command_for_dir(cmd, args[0], &args[1..])?; Ok(code) } -fn which(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - let binary = m.value_of("command").unwrap(); - let binary_path = if m.is_present("toolchain") { - let toolchain = m.value_of("toolchain").unwrap(); +fn which(cfg: &Cfg, m: &ArgMatches) -> Result { + let binary = m.get_one::("command").unwrap(); + let binary_path = if let Some(toolchain) = m.get_one::("toolchain") { cfg.which_binary_by_toolchain(toolchain, binary)? } else { cfg.which_binary(&utils::current_dir()?, binary)? @@ -1068,8 +1090,8 @@ fn which(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn show(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - let verbose = m.is_present("verbose"); +fn show(cfg: &Cfg, m: &ArgMatches) -> Result { + let verbose = m.get_flag("verbose"); // Print host triple { @@ -1229,8 +1251,8 @@ fn show(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn show_active_toolchain(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - let verbose = m.is_present("verbose"); +fn show_active_toolchain(cfg: &Cfg, m: &ArgMatches) -> Result { + let verbose = m.get_flag("verbose"); let cwd = utils::current_dir()?; match cfg.find_or_install_override_toolchain_or_default(&cwd) { Err(e) => { @@ -1261,17 +1283,17 @@ fn show_rustup_home(cfg: &Cfg) -> Result { Ok(utils::ExitCode(0)) } -fn target_list(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { +fn target_list(cfg: &Cfg, m: &ArgMatches) -> Result { let toolchain = explicit_or_dir_toolchain(cfg, m)?; - if m.is_present("installed") { + if m.get_flag("installed") { common::list_installed_targets(&toolchain) } else { common::list_targets(&toolchain) } } -fn target_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { +fn target_add(cfg: &Cfg, m: &ArgMatches) -> Result { let toolchain = explicit_or_dir_toolchain(cfg, m)?; // XXX: long term move this error to cli ? the normal .into doesn't work // because Result here is the wrong sort and expression type ascription @@ -1280,10 +1302,10 @@ fn target_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { // custom toolchains. let distributable = DistributableToolchain::new_for_components(&toolchain)?; - let mut targets: Vec = m - .values_of("target") + let mut targets: Vec<_> = m + .get_many::("target") .unwrap() - .map(ToString::to_string) + .map(ToOwned::to_owned) .collect(); if targets.contains(&"all".to_string()) { @@ -1310,10 +1332,10 @@ fn target_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { } } - for target in &targets { + for target in targets { let new_component = Component::new( "rust-std".to_string(), - Some(TargetTriple::new(target)), + Some(TargetTriple::new(&target)), false, ); distributable.add_component(new_component)?; @@ -1322,10 +1344,10 @@ fn target_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn target_remove(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { +fn target_remove(cfg: &Cfg, m: &ArgMatches) -> Result { let toolchain = explicit_or_dir_toolchain(cfg, m)?; - for target in m.values_of("target").unwrap() { + for target in m.get_many::("target").unwrap() { let new_component = Component::new( "rust-std".to_string(), Some(TargetTriple::new(target)), @@ -1338,10 +1360,10 @@ fn target_remove(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn component_list(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { +fn component_list(cfg: &Cfg, m: &ArgMatches) -> Result { let toolchain = explicit_or_dir_toolchain(cfg, m)?; - if m.is_present("installed") { + if m.get_flag("installed") { common::list_installed_components(&toolchain) } else { common::list_components(&toolchain)?; @@ -1349,18 +1371,22 @@ fn component_list(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { } } -fn component_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { +fn component_add(cfg: &Cfg, m: &ArgMatches) -> Result { let toolchain = explicit_or_dir_toolchain(cfg, m)?; let distributable = DistributableToolchain::new(&toolchain)?; - let target = m.value_of("target").map(TargetTriple::new).or_else(|| { - distributable - .desc() - .as_ref() - .ok() - .map(|desc| desc.target.clone()) - }); - - for component in m.values_of("component").unwrap() { + let target = m + .get_one::("target") + .map(|s| &**s) + .map(TargetTriple::new) + .or_else(|| { + distributable + .desc() + .as_ref() + .ok() + .map(|desc| desc.target.clone()) + }); + + for component in m.get_many::("component").unwrap() { let new_component = Component::new_with_target(component, false) .unwrap_or_else(|| Component::new(component.to_string(), target.clone(), true)); distributable.add_component(new_component)?; @@ -1369,18 +1395,22 @@ fn component_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn component_remove(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { +fn component_remove(cfg: &Cfg, m: &ArgMatches) -> Result { let toolchain = explicit_or_dir_toolchain(cfg, m)?; let distributable = DistributableToolchain::new_for_components(&toolchain)?; - let target = m.value_of("target").map(TargetTriple::new).or_else(|| { - distributable - .desc() - .as_ref() - .ok() - .map(|desc| desc.target.clone()) - }); - - for component in m.values_of("component").unwrap() { + let target = m + .get_one::("target") + .map(|s| &**s) + .map(TargetTriple::new) + .or_else(|| { + distributable + .desc() + .as_ref() + .ok() + .map(|desc| desc.target.clone()) + }); + + for component in m.get_many::("component").unwrap() { let new_component = Component::new_with_target(component, false) .unwrap_or_else(|| Component::new(component.to_string(), target.clone(), true)); distributable.remove_component(new_component)?; @@ -1389,8 +1419,8 @@ fn component_remove(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn explicit_or_dir_toolchain<'a>(cfg: &'a Cfg, m: &ArgMatches<'_>) -> Result> { - let toolchain = m.value_of("toolchain"); +fn explicit_or_dir_toolchain<'a>(cfg: &'a Cfg, m: &ArgMatches) -> Result> { + let toolchain = m.get_one::("toolchain"); if let Some(toolchain) = toolchain { let toolchain = cfg.get_toolchain(toolchain, false)?; return Ok(toolchain); @@ -1402,13 +1432,13 @@ fn explicit_or_dir_toolchain<'a>(cfg: &'a Cfg, m: &ArgMatches<'_>) -> Result) -> Result { - common::list_toolchains(cfg, m.is_present("verbose")) +fn toolchain_list(cfg: &Cfg, m: &ArgMatches) -> Result { + common::list_toolchains(cfg, m.get_flag("verbose")) } -fn toolchain_link(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - let toolchain = m.value_of("toolchain").unwrap(); - let path = m.value_of("path").unwrap(); +fn toolchain_link(cfg: &Cfg, m: &ArgMatches) -> Result { + let toolchain = m.get_one::("toolchain").unwrap(); + let path = m.get_one::("path").unwrap(); let toolchain = cfg.get_toolchain(toolchain, true)?; if let Ok(custom) = CustomToolchain::new(&toolchain) { @@ -1422,16 +1452,16 @@ fn toolchain_link(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { } } -fn toolchain_remove(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { - for toolchain in m.values_of("toolchain").unwrap() { +fn toolchain_remove(cfg: &mut Cfg, m: &ArgMatches) -> Result { + for toolchain in m.get_many::("toolchain").unwrap() { let toolchain = cfg.get_toolchain(toolchain, false)?; toolchain.remove()?; } Ok(utils::ExitCode(0)) } -fn override_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - let toolchain = m.value_of("toolchain").unwrap(); +fn override_add(cfg: &Cfg, m: &ArgMatches) -> Result { + let toolchain = m.get_one::("toolchain").unwrap(); let toolchain = cfg.get_toolchain(toolchain, false)?; let status = if !toolchain.is_custom() { @@ -1443,7 +1473,7 @@ fn override_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { None }; - let path = if let Some(path) = m.value_of("path") { + let path = if let Some(path) = m.get_one::("path") { PathBuf::from(path) } else { utils::current_dir()? @@ -1458,8 +1488,8 @@ fn override_add(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn override_remove(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - let paths = if m.is_present("nonexistent") { +fn override_remove(cfg: &Cfg, m: &ArgMatches) -> Result { + let paths = if m.get_flag("nonexistent") { let list: Vec<_> = cfg.settings_file.with(|s| { Ok(s.overrides .iter() @@ -1476,8 +1506,8 @@ fn override_remove(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { info!("no nonexistent paths detected"); } list - } else if m.is_present("path") { - vec![m.value_of("path").unwrap().to_string()] + } else if let Some(path) = m.get_one::("path") { + vec![path.to_owned()] } else { vec![utils::current_dir()?.to_str().unwrap().to_string()] }; @@ -1490,7 +1520,7 @@ fn override_remove(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { info!("override toolchain for '{}' removed", path); } else { info!("no override toolchain for '{}'", path); - if !m.is_present("path") && !m.is_present("nonexistent") { + if !m.get_one::("path").is_some() && !m.get_flag("nonexistent") { info!( "you may use `--path ` option to remove override toolchain \ for a specific path" @@ -1501,7 +1531,7 @@ fn override_remove(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -const DOCS_DATA: &[(&str, &str, &str,)] = &[ +const DOCS_DATA: &[(&str, &str, &str)] = &[ // flags can be used to open specific documents, e.g. `rustup doc --nomicon` // tuple elements: document name used as flag, help message, document index path ("alloc", "The Rust core allocation and collections library", "alloc/index.html"), @@ -1521,7 +1551,7 @@ const DOCS_DATA: &[(&str, &str, &str,)] = &[ ("embedded-book", "The Embedded Rust Book", "embedded-book/index.html"), ]; -fn doc(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { +fn doc(cfg: &Cfg, m: &ArgMatches) -> Result { let toolchain = explicit_or_dir_toolchain(cfg, m)?; if let Ok(distributable) = DistributableToolchain::new(&toolchain) { let components = distributable.list_components()?; @@ -1549,16 +1579,16 @@ fn doc(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { } let topical_path: PathBuf; - let doc_url = if let Some(topic) = m.value_of("topic") { + let doc_url = if let Some(topic) = m.get_one::("topic") { topical_path = topical_doc::local_path(&toolchain.doc_path("").unwrap(), topic)?; topical_path.to_str().unwrap() - } else if let Some((_, _, path)) = DOCS_DATA.iter().find(|(name, _, _)| m.is_present(name)) { + } else if let Some((_, _, path)) = DOCS_DATA.iter().find(|(name, _, _)| m.get_flag(name)) { path } else { "index.html" }; - if m.is_present("path") { + if m.get_flag("path") { let doc_path = toolchain.doc_path(doc_url)?; writeln!(process().stdout(), "{}", doc_path.display())?; Ok(utils::ExitCode(0)) @@ -1568,8 +1598,8 @@ fn doc(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { } } -fn man(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - let command = m.value_of("command").unwrap(); +fn man(cfg: &Cfg, m: &ArgMatches) -> Result { + let command = m.get_one::("command").unwrap(); let toolchain = explicit_or_dir_toolchain(cfg, m)?; let mut toolchain = toolchain.path().to_path_buf(); @@ -1582,7 +1612,7 @@ fn man(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { if let Some(path) = process().var_os("MANPATH") { manpaths.push(path); } - Command::new("man") + process::Command::new("man") .env("MANPATH", manpaths) .arg(command) .status() @@ -1590,23 +1620,23 @@ fn man(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { Ok(utils::ExitCode(0)) } -fn self_uninstall(m: &ArgMatches<'_>) -> Result { - let no_prompt = m.is_present("no-prompt"); +fn self_uninstall(m: &ArgMatches) -> Result { + let no_prompt = m.get_flag("no-prompt"); self_update::uninstall(no_prompt) } -fn set_default_host_triple(cfg: &Cfg, m: &ArgMatches<'_>) -> Result { - cfg.set_default_host_triple(m.value_of("host_triple").unwrap())?; +fn set_default_host_triple(cfg: &Cfg, m: &ArgMatches) -> Result { + cfg.set_default_host_triple(m.get_one::("host_triple").unwrap())?; Ok(utils::ExitCode(0)) } -fn set_profile(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { - cfg.set_profile(m.value_of("profile-name").unwrap())?; +fn set_profile(cfg: &mut Cfg, m: &ArgMatches) -> Result { + cfg.set_profile(m.get_one::("profile-name").unwrap())?; Ok(utils::ExitCode(0)) } -fn set_auto_self_update(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result { +fn set_auto_self_update(cfg: &mut Cfg, m: &ArgMatches) -> Result { if self_update::NEVER_SELF_UPDATE { let mut args = crate::process().args_os(); let arg0 = args.next().map(PathBuf::from); @@ -1616,7 +1646,7 @@ fn set_auto_self_update(cfg: &mut Cfg, m: &ArgMatches<'_>) -> Result("auto-self-update-mode").unwrap())?; Ok(utils::ExitCode(0)) } @@ -1640,44 +1670,23 @@ pub(crate) enum CompletionCommand { Cargo, } -static COMPLETIONS: &[(&str, CompletionCommand)] = &[ - ("rustup", CompletionCommand::Rustup), - ("cargo", CompletionCommand::Cargo), -]; - -impl CompletionCommand { - fn variants() -> Vec<&'static str> { - COMPLETIONS.iter().map(|&(s, _)| s).collect::>() +impl clap::ValueEnum for CompletionCommand { + fn value_variants<'a>() -> &'a [Self] { + &[Self::Rustup, Self::Cargo] } -} - -impl FromStr for CompletionCommand { - type Err = String; - fn from_str(s: &str) -> std::result::Result { - match COMPLETIONS - .iter() - .find(|&(val, _)| val.eq_ignore_ascii_case(s)) - { - Some(&(_, cmd)) => Ok(cmd), - None => { - let completion_options = COMPLETIONS - .iter() - .map(|&(v, _)| v) - .fold("".to_owned(), |s, v| format!("{s}{v}, ")); - Err(format!( - "[valid values: {}]", - completion_options.trim_end_matches(", ") - )) - } - } + fn to_possible_value<'a>(&self) -> Option> { + Some(match self { + CompletionCommand::Rustup => PossibleValue::new("rustup"), + CompletionCommand::Cargo => PossibleValue::new("cargo"), + }) } } impl fmt::Display for CompletionCommand { fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { - match COMPLETIONS.iter().find(|&(_, cmd)| cmd == self) { - Some(&(val, _)) => write!(f, "{val}"), + match self.to_possible_value() { + Some(v) => write!(f, "{}", v.get_name()), None => unreachable!(), } } @@ -1686,7 +1695,7 @@ impl fmt::Display for CompletionCommand { fn output_completion_script(shell: Shell, command: CompletionCommand) -> Result { match command { CompletionCommand::Rustup => { - cli().gen_completions_to("rustup", shell, &mut term2::stdout()); + clap_complete::generate(shell, &mut cli(), "rustup", &mut term2::stdout()); } CompletionCommand::Cargo => { if let Shell::Zsh = shell { diff --git a/src/cli/setup_mode.rs b/src/cli/setup_mode.rs index 4fffe99b43..150524a6fc 100644 --- a/src/cli/setup_mode.rs +++ b/src/cli/setup_mode.rs @@ -1,5 +1,5 @@ use anyhow::Result; -use clap::{App, AppSettings, Arg}; +use clap::{builder::PossibleValuesParser, AppSettings, Arg, ArgAction, Command}; use super::common; use super::self_update::{self, InstallOpts}; @@ -22,106 +22,126 @@ pub fn main() -> Result { return Ok(utils::ExitCode(0)); } - // NOTICE: If you change anything here, please make the same changes in rustup-init.sh. - let cli = App::new("rustup-init") + // NOTICE: If you change anything here, please make the same changes in rustup-init.sh + let cli = Command::new("rustup-init") .version(common::version()) .about("The installer for rustup") .setting(AppSettings::DeriveDisplayOrder) + .help_template( + "\ +{name} {version} +{about} + +USAGE: + {usage} + +{all-args}", + ) .arg( - Arg::with_name("verbose") - .short("v") + Arg::new("verbose") + .short('v') .long("verbose") - .help("Enable verbose output"), + .help("Enable verbose output") + .action(ArgAction::SetTrue), ) .arg( - Arg::with_name("quiet") + Arg::new("quiet") .conflicts_with("verbose") - .short("q") + .short('q') .long("quiet") - .help("Disable progress output"), + .help("Disable progress output") + .action(ArgAction::SetTrue), ) .arg( - Arg::with_name("no-prompt") - .short("y") - .help("Disable confirmation prompt."), + Arg::new("no-prompt") + .short('y') + .help("Disable confirmation prompt.") + .action(ArgAction::SetTrue), ) .arg( - Arg::with_name("default-host") + Arg::new("default-host") .long("default-host") .takes_value(true) .help("Choose a default host triple"), ) .arg( - Arg::with_name("default-toolchain") + Arg::new("default-toolchain") .long("default-toolchain") .takes_value(true) - .help("Choose a default toolchain to install. Use 'none' to not install any toolchains at all"), + .help("Choose a default toolchain to install"), ) .arg( - Arg::with_name("profile") + Arg::new("profile") .long("profile") - .possible_values(Profile::names()) + .value_parser(PossibleValuesParser::new(Profile::names())) .default_value(Profile::default_name()), ) .arg( - Arg::with_name("components") + Arg::new("components") .help("Component name to also install") .long("component") - .short("c") + .short('c') .takes_value(true) - .multiple(true) - .use_delimiter(true), + .multiple_values(true) + .use_value_delimiter(true) + .action(ArgAction::Append), ) .arg( - Arg::with_name("targets") + Arg::new("targets") .help("Target name to also install") .long("target") - .short("target") + .short('t') .takes_value(true) - .multiple(true) - .use_delimiter(true), + .multiple_values(true) + .use_value_delimiter(true), ) .arg( - Arg::with_name("no-update-default-toolchain") + Arg::new("no-update-default-toolchain") .long("no-update-default-toolchain") - .help("Don't update any existing default toolchain after install"), + .help("Don't update any existing default toolchain after install") + .action(ArgAction::SetTrue), ) .arg( - Arg::with_name("no-modify-path") + Arg::new("no-modify-path") .long("no-modify-path") - .help("Don't configure the PATH environment variable"), + .help("Don't configure the PATH environment variable") + .action(ArgAction::SetTrue), ); - let matches = match cli.get_matches_from_safe(process().args_os()) { + let matches = match cli.try_get_matches_from(process().args_os()) { Ok(matches) => matches, Err(e) - if e.kind == clap::ErrorKind::HelpDisplayed - || e.kind == clap::ErrorKind::VersionDisplayed => + if e.kind() == clap::ErrorKind::DisplayHelp + || e.kind() == clap::ErrorKind::DisplayVersion => { - writeln!(process().stdout().lock(), "{}", e.message)?; + writeln!(process().stdout().lock(), "{e}")?; return Ok(utils::ExitCode(0)); } Err(e) => return Err(e.into()), }; - let no_prompt = matches.is_present("no-prompt"); - let verbose = matches.is_present("verbose"); - let quiet = matches.is_present("quiet"); - let default_host = matches.value_of("default-host").map(ToOwned::to_owned); - let default_toolchain = matches.value_of("default-toolchain").map(ToOwned::to_owned); + let no_prompt = matches.get_flag("no-prompt"); + let verbose = matches.get_flag("verbose"); + let quiet = matches.get_flag("quiet"); + let default_host = matches + .get_one::("default-host") + .map(ToOwned::to_owned); + let default_toolchain = matches + .get_one::("default-toolchain") + .map(ToOwned::to_owned); let profile = matches - .value_of("profile") + .get_one::("profile") .expect("Unreachable: Clap should supply a default"); - let no_modify_path = matches.is_present("no-modify-path"); - let no_update_toolchain = matches.is_present("no-update-default-toolchain"); + let no_modify_path = matches.get_flag("no-modify-path"); + let no_update_toolchain = matches.get_flag("no-update-default-toolchain"); let components: Vec<_> = matches - .values_of("components") - .map(|v| v.collect()) + .get_many::("components") + .map(|v| v.map(|s| &**s).collect()) .unwrap_or_else(Vec::new); let targets: Vec<_> = matches - .values_of("targets") - .map(|v| v.collect()) + .get_many::("targets") + .map(|v| v.map(|s| &**s).collect()) .unwrap_or_else(Vec::new); let opts = InstallOpts { diff --git a/tests/suite/cli_misc.rs b/tests/suite/cli_misc.rs index 60d68fc237..1849ffc3d2 100644 --- a/tests/suite/cli_misc.rs +++ b/tests/suite/cli_misc.rs @@ -806,11 +806,11 @@ fn completion_bad_shell() { setup(&|config| { config.expect_err( &["rustup", "completions", "fake"], - "error: 'fake' isn't a valid value for ''", + r#"error: "fake" isn't a valid value for ''"#, ); config.expect_err( &["rustup", "completions", "fake", "cargo"], - "error: 'fake' isn't a valid value for ''", + r#"error: "fake" isn't a valid value for ''"#, ); }); } @@ -820,7 +820,7 @@ fn completion_bad_tool() { setup(&|config| { config.expect_err( &["rustup", "completions", "bash", "fake"], - "error: 'fake' isn't a valid value for ''", + r#"error: "fake" isn't a valid value for ''"#, ); }); } @@ -946,7 +946,7 @@ fn override_by_toolchain_on_the_command_line() { ); config.expect_err( &["rustup", "@stable", "which", "rustc"], - "Invalid value for '<+toolchain>': Toolchain overrides must begin with '+'", + r#"Invalid value "@stable" for '<+toolchain>': Toolchain overrides must begin with '+'"#, ); config.expect_stderr_ok( &["rustup", "+stable", "set", "profile", "minimal"], From aa27cd72e6a769aa5199fea1c9721a0794ddc55b Mon Sep 17 00:00:00 2001 From: hi-rustin Date: Tue, 28 Feb 2023 09:38:13 +0800 Subject: [PATCH 2/6] Fix broken test path --- tests/suite/cli_ui.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/suite/cli_ui.rs b/tests/suite/cli_ui.rs index 74c4fe5b1a..3349f81a33 100644 --- a/tests/suite/cli_ui.rs +++ b/tests/suite/cli_ui.rs @@ -8,11 +8,11 @@ fn rustup_ui_doc_text_tests() { // Copy rustup-init to rustup so that the tests can run it. fs::copy(rustup_init, &rustup).unwrap(); t.register_bin("rustup", &rustup); - t.case("tests/cli-ui/rustup/*.toml"); + t.case("tests/suite/cli-ui/rustup/*.toml"); #[cfg(target_os = "windows")] { // On windows, we don't have man command, so skip the test. - t.skip("tests/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml"); + t.skip("tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml"); } } @@ -53,6 +53,6 @@ fn rustup_init_ui_doc_text_tests() { .unwrap(); // Make sure that the help output of rustup-init and rustup-init.sh are the same. - assert!(rustup_init_help_std_out == rustup_init_sh_help_std_out) + assert_eq!(rustup_init_help_std_out, rustup_init_sh_help_std_out) } } From 55c6fdb70b21af306234f5f5d5e599bafaea9acf Mon Sep 17 00:00:00 2001 From: hi-rustin Date: Thu, 2 Mar 2023 11:14:46 +0800 Subject: [PATCH 3/6] Update setup mode tests --- rustup-init.sh | 48 +++++++++++++------ src/cli/setup_mode.rs | 15 ++---- .../rustup-init_help_flag_stdout.toml | 48 +++++++++++++------ .../rustup-init_sh_help_flag_stdout.toml | 48 +++++++++++++------ 4 files changed, 105 insertions(+), 54 deletions(-) diff --git a/rustup-init.sh b/rustup-init.sh index 3bf9ca969f..e9cfb2f9cf 100755 --- a/rustup-init.sh +++ b/rustup-init.sh @@ -29,25 +29,45 @@ rustup-init 1.25.2 (8c4dad73d 2023-02-01) The installer for rustup USAGE: - rustup-init [FLAGS] [OPTIONS] - -FLAGS: - -v, --verbose Enable verbose output - -q, --quiet Disable progress output - -y Disable confirmation prompt. - --no-update-default-toolchain Don't update any existing default toolchain after install - --no-modify-path Don't configure the PATH environment variable - -h, --help Prints help information - -V, --version Prints version information + rustup-init [OPTIONS] OPTIONS: - --default-host Choose a default host triple + -v, --verbose + Enable verbose output + + -q, --quiet + Disable progress output + + -y + Disable confirmation prompt. + + --default-host + Choose a default host triple + --default-toolchain Choose a default toolchain to install. Use 'none' to not install any toolchains at all - --profile [default: default] [possible values: minimal, default, complete] - -c, --component ... Component name to also install - -t, --target ... Target name to also install + --profile + [default: default] [possible values: minimal, default, complete] + + -c, --component ... + Component name to also install + + -t, --target ... + Target name to also install + + --no-update-default-toolchain + Don't update any existing default toolchain after install + + --no-modify-path + Don't configure the PATH environment variable + + -h, --help + Print help information + + -V, --version + Print version information + EOF } diff --git a/src/cli/setup_mode.rs b/src/cli/setup_mode.rs index 150524a6fc..25e4ecc258 100644 --- a/src/cli/setup_mode.rs +++ b/src/cli/setup_mode.rs @@ -27,16 +27,6 @@ pub fn main() -> Result { .version(common::version()) .about("The installer for rustup") .setting(AppSettings::DeriveDisplayOrder) - .help_template( - "\ -{name} {version} -{about} - -USAGE: - {usage} - -{all-args}", - ) .arg( Arg::new("verbose") .short('v') @@ -68,7 +58,7 @@ USAGE: Arg::new("default-toolchain") .long("default-toolchain") .takes_value(true) - .help("Choose a default toolchain to install"), + .help("Choose a default toolchain to install. Use 'none' to not install any toolchains at all"), ) .arg( Arg::new("profile") @@ -93,7 +83,8 @@ USAGE: .short('t') .takes_value(true) .multiple_values(true) - .use_value_delimiter(true), + .use_value_delimiter(true) + .action(ArgAction::Append), ) .arg( Arg::new("no-update-default-toolchain") diff --git a/tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml b/tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml index c39da7a4a4..1f67e80680 100644 --- a/tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml @@ -6,24 +6,44 @@ rustup-init [..] The installer for rustup USAGE: - rustup-init[EXE] [FLAGS] [OPTIONS] - -FLAGS: - -v, --verbose Enable verbose output - -q, --quiet Disable progress output - -y Disable confirmation prompt. - --no-update-default-toolchain Don't update any existing default toolchain after install - --no-modify-path Don't configure the PATH environment variable - -h, --help Prints help information - -V, --version Prints version information + rustup-init[EXE] [OPTIONS] OPTIONS: - --default-host Choose a default host triple + -v, --verbose + Enable verbose output + + -q, --quiet + Disable progress output + + -y + Disable confirmation prompt. + + --default-host + Choose a default host triple + --default-toolchain Choose a default toolchain to install. Use 'none' to not install any toolchains at all - --profile [default: default] [possible values: minimal, default, complete] - -c, --component ... Component name to also install - -t, --target ... Target name to also install + --profile + [default: default] [possible values: minimal, default, complete] + + -c, --component ... + Component name to also install + + -t, --target ... + Target name to also install + + --no-update-default-toolchain + Don't update any existing default toolchain after install + + --no-modify-path + Don't configure the PATH environment variable + + -h, --help + Print help information + + -V, --version + Print version information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml b/tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml index a84f663992..165e444002 100644 --- a/tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml @@ -6,24 +6,44 @@ rustup-init [..] The installer for rustup USAGE: - rustup-init[EXE] [FLAGS] [OPTIONS] - -FLAGS: - -v, --verbose Enable verbose output - -q, --quiet Disable progress output - -y Disable confirmation prompt. - --no-update-default-toolchain Don't update any existing default toolchain after install - --no-modify-path Don't configure the PATH environment variable - -h, --help Prints help information - -V, --version Prints version information + rustup-init[EXE] [OPTIONS] OPTIONS: - --default-host Choose a default host triple + -v, --verbose + Enable verbose output + + -q, --quiet + Disable progress output + + -y + Disable confirmation prompt. + + --default-host + Choose a default host triple + --default-toolchain Choose a default toolchain to install. Use 'none' to not install any toolchains at all - --profile [default: default] [possible values: minimal, default, complete] - -c, --component ... Component name to also install - -t, --target ... Target name to also install + --profile + [default: default] [possible values: minimal, default, complete] + + -c, --component ... + Component name to also install + + -t, --target ... + Target name to also install + + --no-update-default-toolchain + Don't update any existing default toolchain after install + + --no-modify-path + Don't configure the PATH environment variable + + -h, --help + Print help information + + -V, --version + Print version information + """ stderr = "" From 7e4656a7b22dd2b8e6f97d6e55f4e2c083498dec Mon Sep 17 00:00:00 2001 From: hi-rustin Date: Fri, 3 Mar 2023 16:09:37 +0800 Subject: [PATCH 4/6] Update rustup mode tests --- src/cli/rustup_mode.rs | 25 +++------ .../rustup_check_cmd_help_flag_stdout.toml | 5 +- ...stup_completions_cmd_help_flag_stdout.toml | 11 ++-- ...omponent_cmd_add_cmd_help_flag_stdout.toml | 11 ++-- ...mponent_cmd_list_cmd_help_flag_stdout.toml | 13 +++-- ...onent_cmd_remove_cmd_help_flag_stdout.toml | 11 ++-- .../rustup_default_cmd_help_flag_stdout.toml | 11 ++-- .../rustup_doc_cmd_help_flag_stdout.toml | 53 ++++++++++--------- .../cli-ui/rustup/rustup_help_cmd_stdout.toml | 17 +++--- .../rustup/rustup_help_flag_stdout.toml | 17 +++--- .../rustup_man_cmd_help_flag_stdout.toml | 11 ++-- ...override_cmd_add_cmd_help_flag_stdout.toml | 9 ++-- .../rustup_override_cmd_help_flag_stdout.toml | 7 +-- ...verride_cmd_list_cmd_help_flag_stdout.toml | 5 +- ...rride_cmd_remove_cmd_help_flag_stdout.toml | 9 ++-- ...override_cmd_set_cmd_help_flag_stdout.toml | 9 ++-- ...erride_cmd_unset_cmd_help_flag_stdout.toml | 9 ++-- .../rustup_run_cmd_help_flag_stdout.toml | 15 +++--- .../rustup_self_cmd_help_flag_stdout.toml | 7 +-- ...lf_cmd_uninstall_cmd_help_flag_stdout.toml | 7 +-- ..._self_cmd_update_cmd_help_flag_stdout.toml | 5 +- ...md_upgrade-data _cmd_help_flag_stdout.toml | 5 +- ...auto-self-update_cmd_help_flag_stdout.toml | 10 ++-- ...cmd_default-host_cmd_help_flag_stdout.toml | 8 +-- .../rustup_set_cmd_help_flag_stdout.toml | 9 ++-- ..._set_cmd_profile_cmd_help_flag_stdout.toml | 10 ++-- ...active-toolchain_cmd_help_flag_stdout.toml | 7 +-- .../rustup_show_cmd_help_flag_stdout.toml | 9 ++-- ...up_show_cmd_home_cmd_help_flag_stdout.toml | 5 +- ...up_show_cmd_keys_cmd_help_flag_stdout.toml | 5 +- ...show_cmd_profile_cmd_help_flag_stdout.toml | 5 +- ...p_target_cmd_add_cmd_help_flag_stdout.toml | 11 ++-- .../rustup_target_cmd_help_flag_stdout.toml | 7 +-- ..._target_cmd_list_cmd_help_flag_stdout.toml | 13 +++-- ...arget_cmd_remove_cmd_help_flag_stdout.toml | 11 ++-- ...rustup_toolchain_cmd_help_flag_stdout.toml | 7 +-- ...hain_cmd_install_cmd_help_flag_stdout.toml | 25 ++++----- ...olchain_cmd_link_cmd_help_flag_stdout.toml | 7 +-- ...olchain_cmd_list_cmd_help_flag_stdout.toml | 7 +-- ...in_cmd_uninstall_cmd_help_flag_stdout.toml | 11 ++-- .../rustup_up_cmd_help_flag_stdout.toml | 21 ++++---- .../rustup_update_cmd_help_flag_stdout.toml | 21 ++++---- .../rustup_upgrade_cmd_help_flag_stdout.toml | 21 ++++---- .../rustup_which_cmd_help_flag_stdout.toml | 11 ++-- 44 files changed, 259 insertions(+), 254 deletions(-) diff --git a/src/cli/rustup_mode.rs b/src/cli/rustup_mode.rs index c856f11833..c6f8550d36 100644 --- a/src/cli/rustup_mode.rs +++ b/src/cli/rustup_mode.rs @@ -71,11 +71,11 @@ pub fn main() -> Result { let matches = match cli().try_get_matches_from(process().args_os()) { Ok(matches) => Ok(matches), Err(err) if err.kind() == DisplayHelp => { - writeln!(process().stdout().lock(), "{}", err)?; + writeln!(process().stdout().lock(), "{err}")?; return Ok(utils::ExitCode(0)); } Err(err) if err.kind() == DisplayVersion => { - writeln!(process().stdout().lock(), "{}", err)?; + writeln!(process().stdout().lock(), "{err}")?; info!("This is the version for the rustup toolchain manager, not the rustc compiler."); fn rustc_version() -> std::result::Result> { @@ -107,7 +107,7 @@ pub fn main() -> Result { ] .contains(&err.kind()) { - writeln!(process().stdout().lock(), "{}", err)?; + writeln!(process().stdout().lock(), "{err}")?; return Ok(utils::ExitCode(1)); } Err(err) @@ -207,16 +207,6 @@ pub(crate) fn cli() -> Command<'static> { .arg( verbose_arg("Enable verbose output"), ) - .help_template( - "\ -{name} {version} -{about} - -USAGE: - {usage} - -{all-args}", - ) .arg( Arg::new("quiet") .conflicts_with("verbose") @@ -351,7 +341,6 @@ USAGE: Arg::new("toolchain") .help(TOOLCHAIN_ARG_HELP) .required(false) - .takes_value(true), ), ) .subcommand( @@ -392,7 +381,7 @@ USAGE: .takes_value(true) .multiple_values(true) .use_value_delimiter(true) - .action(ArgAction::Append), + .action(ArgAction::Append), ) .arg( Arg::new("targets") @@ -401,8 +390,8 @@ USAGE: .short('t') .takes_value(true) .multiple_values(true) - .multiple_occurrences(true) - .use_value_delimiter(true), + .use_value_delimiter(true) + .action(ArgAction::Append), ) .arg( Arg::new("no-self-update") @@ -1520,7 +1509,7 @@ fn override_remove(cfg: &Cfg, m: &ArgMatches) -> Result { info!("override toolchain for '{}' removed", path); } else { info!("no override toolchain for '{}'", path); - if !m.get_one::("path").is_some() && !m.get_flag("nonexistent") { + if m.get_one::("path").is_none() && !m.get_flag("nonexistent") { info!( "you may use `--path ` option to remove override toolchain \ for a specific path" diff --git a/tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml index 3e707294c6..0be13627b2 100644 --- a/tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml @@ -7,7 +7,8 @@ Check for updates to Rust toolchains and rustup USAGE: rustup[EXE] check -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml index 0389eea1fd..c4f31ff299 100644 --- a/tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml @@ -7,12 +7,12 @@ Generate tab-completion scripts for your shell USAGE: rustup[EXE] completions [ARGS] -FLAGS: - -h, --help Prints help information - ARGS: - [possible values: zsh, bash, fish, powershell, elvish] - [possible values: rustup, cargo] + [possible values: bash, elvish, fish, powershell, zsh] + [possible values: rustup, cargo] + +OPTIONS: + -h, --help Print help information DISCUSSION: Enable tab completion for Bash, Fish, Zsh, or PowerShell @@ -134,5 +134,6 @@ ${env:USERPROFILE}/Documents/WindowsPowerShell/Microsoft.PowerShell_profile.ps1 ZSH: $ rustup completions zsh cargo > ~/.zfunc/_cargo + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml index 8d3af97ad6..cb5f15b2b7 100644 --- a/tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml @@ -7,15 +7,14 @@ Add a component to a Rust toolchain USAGE: rustup[EXE] component add [OPTIONS] ... -FLAGS: - -h, --help Prints help information +ARGS: + ... OPTIONS: + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` --target - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` + -h, --help Print help information -ARGS: - ... """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml index b4c64893fe..cb1516719a 100644 --- a/tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml @@ -5,14 +5,13 @@ stdout = """ List installed and available components USAGE: - rustup[EXE] component list [FLAGS] [OPTIONS] - -FLAGS: - -h, --help Prints help information - --installed List only installed components + rustup[EXE] component list [OPTIONS] OPTIONS: - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + --installed List only installed components + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml index eafe8eb470..3a84f7092c 100644 --- a/tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml @@ -7,15 +7,14 @@ Remove a component from a Rust toolchain USAGE: rustup[EXE] component remove [OPTIONS] ... -FLAGS: - -h, --help Prints help information +ARGS: + ... OPTIONS: + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` --target - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` + -h, --help Print help information -ARGS: - ... """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml index 86e86a477a..1bb3bd154c 100644 --- a/tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml @@ -7,15 +7,16 @@ Set the default toolchain USAGE: rustup[EXE] default [toolchain] -FLAGS: - -h, --help Prints help information - ARGS: - Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` + Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information + see `rustup help toolchain` + +OPTIONS: + -h, --help Print help information DISCUSSION: Sets the default toolchain to the one specified. If the toolchain is not already installed then it is installed first. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml index 90e88d4d5e..69e11f2c6a 100644 --- a/tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml @@ -5,35 +5,35 @@ stdout = """ Open the documentation for the current toolchain USAGE: - rustup[EXE] doc [FLAGS] [OPTIONS] [topic] + rustup[EXE] doc [OPTIONS] [topic] -FLAGS: - --alloc The Rust core allocation and collections library - --book The Rust Programming Language book - --cargo The Cargo Book - --core The Rust Core Library - --edition-guide The Rust Edition Guide - --embedded-book The Embedded Rust Book - -h, --help Prints help information - --nomicon The Dark Arts of Advanced and Unsafe Rust Programming - --path Only print the path to the documentation - --proc_macro A support library for macro authors when defining new macros - --reference The Rust Reference - --rust-by-example A collection of runnable examples that illustrate various Rust concepts and standard - libraries - --rustc The compiler for the Rust programming language - --rustdoc Documentation generator for Rust projects - --std Standard library API documentation - --test Support code for rustc's built in unit-test and micro-benchmarking framework - --unstable-book The Unstable Book +ARGS: + Topic such as 'core', 'fn', 'usize', 'eprintln!', 'core::arch', 'alloc::format!', + 'std::fs', 'std::fs::read_dir', 'std::io::Bytes', 'std::iter::Sum', + 'std::io::error::Result' etc... OPTIONS: - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` - -ARGS: - Topic such as 'core', 'fn', 'usize', 'eprintln!', 'core::arch', 'alloc::format!', 'std::fs', - 'std::fs::read_dir', 'std::io::Bytes', 'std::iter::Sum', 'std::io::error::Result' etc... + --path Only print the path to the documentation + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + --alloc The Rust core allocation and collections library + --book The Rust Programming Language book + --cargo The Cargo Book + --core The Rust Core Library + --edition-guide The Rust Edition Guide + --nomicon The Dark Arts of Advanced and Unsafe Rust Programming + --proc_macro A support library for macro authors when defining new macros + --reference The Rust Reference + --rust-by-example A collection of runnable examples that illustrate various Rust + concepts and standard libraries + --rustc The compiler for the Rust programming language + --rustdoc Documentation generator for Rust projects + --std Standard library API documentation + --test Support code for rustc's built in unit-test and + micro-benchmarking framework + --unstable-book The Unstable Book + --embedded-book The Embedded Rust Book + -h, --help Print help information DISCUSSION: Opens the documentation for the currently active toolchain with @@ -41,5 +41,6 @@ DISCUSSION: By default, it opens the documentation index. Use the various flags to open specific pieces of documentation. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml b/tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml index 2d37691dd6..05c1c7a523 100644 --- a/tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml @@ -6,17 +6,17 @@ rustup [..] The Rust toolchain installer USAGE: - rustup[EXE] [FLAGS] [+toolchain] - -FLAGS: - -v, --verbose Enable verbose output - -q, --quiet Disable progress output - -h, --help Prints help information - -V, --version Prints version information + rustup[EXE] [OPTIONS] [+toolchain] [SUBCOMMAND] ARGS: <+toolchain> release channel (e.g. +stable) or custom toolchain to set override +OPTIONS: + -v, --verbose Enable verbose output + -q, --quiet Disable progress output + -h, --help Print help information + -V, --version Print version information + SUBCOMMANDS: show Show the active and installed toolchains or profiles update Update Rust toolchains and rustup @@ -33,7 +33,7 @@ SUBCOMMANDS: self Modify the rustup installation set Alter rustup settings completions Generate tab-completion scripts for your shell - help Prints this message or the help of the given subcommand(s) + help Print this message or the help of the given subcommand(s) DISCUSSION: Rustup installs The Rust Programming Language from the official @@ -44,5 +44,6 @@ DISCUSSION: If you are new to Rust consider running `rustup doc --book` to learn Rust. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml index 3a53e3ea37..0964579015 100644 --- a/tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml @@ -6,17 +6,17 @@ rustup [..] The Rust toolchain installer USAGE: - rustup[EXE] [FLAGS] [+toolchain] - -FLAGS: - -v, --verbose Enable verbose output - -q, --quiet Disable progress output - -h, --help Prints help information - -V, --version Prints version information + rustup[EXE] [OPTIONS] [+toolchain] [SUBCOMMAND] ARGS: <+toolchain> release channel (e.g. +stable) or custom toolchain to set override +OPTIONS: + -v, --verbose Enable verbose output + -q, --quiet Disable progress output + -h, --help Print help information + -V, --version Print version information + SUBCOMMANDS: show Show the active and installed toolchains or profiles update Update Rust toolchains and rustup @@ -33,7 +33,7 @@ SUBCOMMANDS: self Modify the rustup installation set Alter rustup settings completions Generate tab-completion scripts for your shell - help Prints this message or the help of the given subcommand(s) + help Print this message or the help of the given subcommand(s) DISCUSSION: Rustup installs The Rust Programming Language from the official @@ -44,5 +44,6 @@ DISCUSSION: If you are new to Rust consider running `rustup doc --book` to learn Rust. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml index 8b40dcc950..962851a0b5 100644 --- a/tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml @@ -7,14 +7,13 @@ View the man page for a given command USAGE: rustup[EXE] man [OPTIONS] -FLAGS: - -h, --help Prints help information +ARGS: + OPTIONS: - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + -h, --help Print help information -ARGS: - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_add_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_add_cmd_help_flag_stdout.toml index 9b60d6d6ce..b5bd0f793f 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_add_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_add_cmd_help_flag_stdout.toml @@ -7,14 +7,13 @@ Set the override toolchain for a directory USAGE: rustup[EXE] override set [OPTIONS] -FLAGS: - -h, --help Prints help information +ARGS: + Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information + see `rustup help toolchain` OPTIONS: --path Path to the directory + -h, --help Print help information -ARGS: - Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml index 29611950c2..e99dedf9a9 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml @@ -7,14 +7,14 @@ Modify directory toolchain overrides USAGE: rustup[EXE] override -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information SUBCOMMANDS: list List directory toolchain overrides set Set the override toolchain for a directory unset Remove the override toolchain for a directory - help Prints this message or the help of the given subcommand(s) + help Print this message or the help of the given subcommand(s) DISCUSSION: Overrides configure Rustup to use a specific toolchain when @@ -36,5 +36,6 @@ DISCUSSION: To see the active toolchain use `rustup show`. To remove the override and use the default toolchain again, `rustup override unset`. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_list_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_list_cmd_help_flag_stdout.toml index e1c8952087..7aeba37d02 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_list_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_list_cmd_help_flag_stdout.toml @@ -7,7 +7,8 @@ List directory toolchain overrides USAGE: rustup[EXE] override list -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_remove_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_remove_cmd_help_flag_stdout.toml index 0663165a39..254a9e9b7d 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_remove_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_remove_cmd_help_flag_stdout.toml @@ -5,14 +5,12 @@ stdout = """ Remove the override toolchain for a directory USAGE: - rustup[EXE] override unset [FLAGS] [OPTIONS] - -FLAGS: - -h, --help Prints help information - --nonexistent Remove override toolchain for all nonexistent directories + rustup[EXE] override unset [OPTIONS] OPTIONS: --path Path to the directory + --nonexistent Remove override toolchain for all nonexistent directories + -h, --help Print help information DISCUSSION: If `--path` argument is present, removes the override toolchain @@ -20,5 +18,6 @@ DISCUSSION: present, removes the override toolchain for all nonexistent directories. Otherwise, removes the override toolchain for the current directory. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_set_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_set_cmd_help_flag_stdout.toml index 2447565140..3d669d3801 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_set_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_set_cmd_help_flag_stdout.toml @@ -7,14 +7,13 @@ Set the override toolchain for a directory USAGE: rustup[EXE] override set [OPTIONS] -FLAGS: - -h, --help Prints help information +ARGS: + Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information + see `rustup help toolchain` OPTIONS: --path Path to the directory + -h, --help Print help information -ARGS: - Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_unset_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_unset_cmd_help_flag_stdout.toml index 0663165a39..254a9e9b7d 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_unset_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_unset_cmd_help_flag_stdout.toml @@ -5,14 +5,12 @@ stdout = """ Remove the override toolchain for a directory USAGE: - rustup[EXE] override unset [FLAGS] [OPTIONS] - -FLAGS: - -h, --help Prints help information - --nonexistent Remove override toolchain for all nonexistent directories + rustup[EXE] override unset [OPTIONS] OPTIONS: --path Path to the directory + --nonexistent Remove override toolchain for all nonexistent directories + -h, --help Print help information DISCUSSION: If `--path` argument is present, removes the override toolchain @@ -20,5 +18,6 @@ DISCUSSION: present, removes the override toolchain for all nonexistent directories. Otherwise, removes the override toolchain for the current directory. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml index 0a0c902787..20e1ee1a6b 100644 --- a/tests/suite/cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml @@ -5,17 +5,17 @@ stdout = """ Run a command with an environment configured for a given toolchain USAGE: - rustup[EXE] run [FLAGS] ... - -FLAGS: - -h, --help Prints help information - --install Install the requested toolchain if needed + rustup[EXE] run [OPTIONS] ... ARGS: - Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` + Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` ... +OPTIONS: + --install Install the requested toolchain if needed + -h, --help Print help information + DISCUSSION: Configures an environment to use the given toolchain and then runs the specified program. The command may be any program, not just @@ -30,5 +30,6 @@ DISCUSSION: $ cargo +nightly build $ rustup run nightly cargo build + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml index a60e056fa8..cf87e125d0 100644 --- a/tests/suite/cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml @@ -7,13 +7,14 @@ Modify the rustup installation USAGE: rustup[EXE] self -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information SUBCOMMANDS: update Download and install updates to rustup uninstall Uninstall rustup. upgrade-data Upgrade the internal data format. - help Prints this message or the help of the given subcommand(s) + help Print this message or the help of the given subcommand(s) + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml index 3f230e6496..add410f53a 100644 --- a/tests/suite/cli-ui/rustup/rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml @@ -5,10 +5,11 @@ stdout = """ Uninstall rustup. USAGE: - rustup[EXE] self uninstall [FLAGS] + rustup[EXE] self uninstall [OPTIONS] -FLAGS: - -h, --help Prints help information +OPTIONS: -y + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml index 72cb5f837d..e81a54107b 100644 --- a/tests/suite/cli-ui/rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml @@ -7,7 +7,8 @@ Download and install updates to rustup USAGE: rustup[EXE] self update -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml index 9cf019ea92..a9b9c02234 100644 --- a/tests/suite/cli-ui/rustup/rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml @@ -7,7 +7,8 @@ Upgrade the internal data format. USAGE: rustup[EXE] self upgrade-data -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml index 622ab90522..65508bbbc3 100644 --- a/tests/suite/cli-ui/rustup/rustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml @@ -7,11 +7,11 @@ The rustup auto self update mode USAGE: rustup[EXE] set auto-self-update -FLAGS: - -h, --help Prints help information - -V, --version Prints version information - ARGS: - [default: enable] [possible values: enable, disable, check-only] + [default: enable] [possible values: enable, disable, check-only] + +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_set_cmd_default-host_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_set_cmd_default-host_cmd_help_flag_stdout.toml index 4df7a55911..428953988f 100644 --- a/tests/suite/cli-ui/rustup/rustup_set_cmd_default-host_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_set_cmd_default-host_cmd_help_flag_stdout.toml @@ -7,11 +7,11 @@ The triple used to identify toolchains when not specified USAGE: rustup[EXE] set default-host -FLAGS: - -h, --help Prints help information - -V, --version Prints version information - ARGS: + +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml index 2c92922e19..20641c5c65 100644 --- a/tests/suite/cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml @@ -7,13 +7,14 @@ Alter rustup settings USAGE: rustup[EXE] set -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information SUBCOMMANDS: - auto-self-update The rustup auto self update mode default-host The triple used to identify toolchains when not specified - help Prints this message or the help of the given subcommand(s) profile The default components installed + auto-self-update The rustup auto self update mode + help Print this message or the help of the given subcommand(s) + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml index 1a6db002d1..de26eb61fd 100644 --- a/tests/suite/cli-ui/rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml @@ -7,11 +7,11 @@ The default components installed USAGE: rustup[EXE] set profile -FLAGS: - -h, --help Prints help information - -V, --version Prints version information - ARGS: - [default: default] [possible values: minimal, default, complete] + [default: default] [possible values: minimal, default, complete] + +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml index 93aca775d2..1c8eca7e30 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml @@ -5,11 +5,11 @@ stdout = """ Show the active toolchain USAGE: - rustup[EXE] show active-toolchain [FLAGS] + rustup[EXE] show active-toolchain [OPTIONS] -FLAGS: - -h, --help Prints help information +OPTIONS: -v, --verbose Enable verbose output with rustc information + -h, --help Print help information DISCUSSION: Shows the name of the active toolchain. @@ -19,5 +19,6 @@ DISCUSSION: You should use `rustc --print sysroot` to get the sysroot, or `rustc --version` to get the toolchain version. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml index bb5425ca45..f99000e07c 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml @@ -5,18 +5,18 @@ stdout = """ Show the active and installed toolchains or profiles USAGE: - rustup[EXE] show [FLAGS] [SUBCOMMAND] + rustup[EXE] show [OPTIONS] [SUBCOMMAND] -FLAGS: +OPTIONS: -v, --verbose Enable verbose output with rustc information for all installed toolchains - -h, --help Prints help information + -h, --help Print help information SUBCOMMANDS: active-toolchain Show the active toolchain home Display the computed value of RUSTUP_HOME profile Show the current profile keys Display the known PGP keys - help Prints this message or the help of the given subcommand(s) + help Print this message or the help of the given subcommand(s) DISCUSSION: Shows the name of the active toolchain and the version of `rustc`. @@ -26,5 +26,6 @@ DISCUSSION: If there are multiple toolchains installed then all installed toolchains are listed as well. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml index 130dd5b45a..d00738631e 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml @@ -7,7 +7,8 @@ Display the computed value of RUSTUP_HOME USAGE: rustup[EXE] show home -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml index 752b2bacee..fd0d2dee99 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml @@ -7,7 +7,8 @@ Display the known PGP keys USAGE: rustup[EXE] show keys -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_profile_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_profile_cmd_help_flag_stdout.toml index 1e6995f092..49dbe0213b 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_profile_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_profile_cmd_help_flag_stdout.toml @@ -7,7 +7,8 @@ Show the current profile USAGE: rustup[EXE] show profile -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml index 5eb8a3b096..fea2874f81 100644 --- a/tests/suite/cli-ui/rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml @@ -7,14 +7,13 @@ Add a target to a Rust toolchain USAGE: rustup[EXE] target add [OPTIONS] ... -FLAGS: - -h, --help Prints help information +ARGS: + ... List of targets to install; \"all\" installs all available targets OPTIONS: - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + -h, --help Print help information -ARGS: - ... List of targets to install; \"all\" installs all available targets """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml index a11997fdf9..4948b5e0da 100644 --- a/tests/suite/cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml @@ -7,13 +7,14 @@ Modify a toolchain's supported targets USAGE: rustup[EXE] target -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information SUBCOMMANDS: list List installed and available targets add Add a target to a Rust toolchain remove Remove a target from a Rust toolchain - help Prints this message or the help of the given subcommand(s) + help Print this message or the help of the given subcommand(s) + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml index ae6df3b036..e8a424e714 100644 --- a/tests/suite/cli-ui/rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml @@ -5,14 +5,13 @@ stdout = """ List installed and available targets USAGE: - rustup[EXE] target list [FLAGS] [OPTIONS] - -FLAGS: - -h, --help Prints help information - --installed List only installed targets + rustup[EXE] target list [OPTIONS] OPTIONS: - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + --installed List only installed targets + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_target_cmd_remove_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_target_cmd_remove_cmd_help_flag_stdout.toml index c66298321f..6fecca9ccf 100644 --- a/tests/suite/cli-ui/rustup/rustup_target_cmd_remove_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_target_cmd_remove_cmd_help_flag_stdout.toml @@ -7,14 +7,13 @@ Remove a target from a Rust toolchain USAGE: rustup[EXE] target remove [OPTIONS] ... -FLAGS: - -h, --help Prints help information +ARGS: + ... List of targets to uninstall OPTIONS: - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + -h, --help Print help information -ARGS: - ... """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_help_flag_stdout.toml index eed7045818..6b81d52fba 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_help_flag_stdout.toml @@ -7,15 +7,15 @@ Modify or query the installed toolchains USAGE: rustup[EXE] toolchain -FLAGS: - -h, --help Prints help information +OPTIONS: + -h, --help Print help information SUBCOMMANDS: list List installed toolchains install Install or update a given toolchain uninstall Uninstall a toolchain link Create a custom toolchain by symlinking to a directory - help Prints this message or the help of the given subcommand(s) + help Print this message or the help of the given subcommand(s) DISCUSSION: Many `rustup` commands deal with *toolchains*, a single @@ -58,5 +58,6 @@ DISCUSSION: rustup can also manage symlinked local toolchain builds, which are often used for developing Rust itself. For more information see `rustup toolchain help link`. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml index 2e4253449f..ae33af4b39 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml @@ -5,23 +5,24 @@ stdout = """ Install or update a given toolchain USAGE: - rustup[EXE] toolchain install [FLAGS] [OPTIONS] ... + rustup[EXE] toolchain install [OPTIONS] ... -FLAGS: - --allow-downgrade Allow rustup to downgrade the toolchain to satisfy your component choice - --force Force an update, even if some components are missing - --force-non-host Install toolchains that require an emulator. See https://github.com/rust- - lang/rustup/wiki/Non-host-toolchains - -h, --help Prints help information - --no-self-update Don't perform self update when running the`rustup toolchain install` command +ARGS: + ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` OPTIONS: + --profile [possible values: minimal, default, complete] -c, --component ... Add specific components on installation - --profile [possible values: minimal, default, complete] -t, --target ... Add specific targets on installation + --no-self-update Don't perform self update when running the`rustup toolchain + install` command + --force Force an update, even if some components are missing + --allow-downgrade Allow rustup to downgrade the toolchain to satisfy your + component choice + --force-non-host Install toolchains that require an emulator. See + https://github.com/rust-lang/rustup/wiki/Non-host-toolchains + -h, --help Print help information -ARGS: - ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml index 82e809fe4c..31d023c567 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml @@ -7,13 +7,13 @@ Create a custom toolchain by symlinking to a directory USAGE: rustup[EXE] toolchain link -FLAGS: - -h, --help Prints help information - ARGS: Custom toolchain name Path to the directory +OPTIONS: + -h, --help Print help information + DISCUSSION: 'toolchain' is the custom name to be assigned to the new toolchain. Any name is permitted as long as it does not fully match an initial @@ -32,5 +32,6 @@ DISCUSSION: If you now compile a crate in the current directory, the custom toolchain 'latest-stage1' will be used. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml index b40f471f0f..84db5f94e3 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml @@ -5,10 +5,11 @@ stdout = """ List installed toolchains USAGE: - rustup[EXE] toolchain list [FLAGS] + rustup[EXE] toolchain list [OPTIONS] -FLAGS: - -h, --help Prints help information +OPTIONS: -v, --verbose Enable verbose output with toolchain information + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml index b6c58490e2..c88f683251 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml @@ -7,11 +7,12 @@ Uninstall a toolchain USAGE: rustup[EXE] toolchain uninstall ... -FLAGS: - -h, --help Prints help information - ARGS: - ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` + ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + +OPTIONS: + -h, --help Print help information + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml index eed4674058..8122173b68 100644 --- a/tests/suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml @@ -5,18 +5,18 @@ stdout = """ Update Rust toolchains and rustup USAGE: - rustup[EXE] update [FLAGS] [toolchain]... - -FLAGS: - --force Force an update, even if some components are missing - --force-non-host Install toolchains that require an emulator. See https://github.com/rust- - lang/rustup/wiki/Non-host-toolchains - -h, --help Prints help information - --no-self-update Don't perform self update when running the `rustup update` command + rustup[EXE] update [OPTIONS] [toolchain]... ARGS: - ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` + ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + +OPTIONS: + --no-self-update Don't perform self update when running the `rustup update` command + --force Force an update, even if some components are missing + --force-non-host Install toolchains that require an emulator. See + https://github.com/rust-lang/rustup/wiki/Non-host-toolchains + -h, --help Print help information DISCUSSION: With no toolchain specified, the `update` command updates each of @@ -25,5 +25,6 @@ DISCUSSION: If given a toolchain argument then `update` updates that toolchain, the same as `rustup toolchain install`. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml index 9d6c272728..166a9f5dba 100644 --- a/tests/suite/cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml @@ -5,18 +5,18 @@ stdout = """ Update Rust toolchains and rustup USAGE: - rustup[EXE] update [FLAGS] [toolchain]... - -FLAGS: - --force Force an update, even if some components are missing - --force-non-host Install toolchains that require an emulator. See https://github.com/rust- - lang/rustup/wiki/Non-host-toolchains - -h, --help Prints help information - --no-self-update Don't perform self update when running the `rustup update` command + rustup[EXE] update [OPTIONS] [toolchain]... ARGS: - ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` + ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + +OPTIONS: + --no-self-update Don't perform self update when running the `rustup update` command + --force Force an update, even if some components are missing + --force-non-host Install toolchains that require an emulator. See + https://github.com/rust-lang/rustup/wiki/Non-host-toolchains + -h, --help Print help information DISCUSSION: With no toolchain specified, the `update` command updates each of @@ -25,5 +25,6 @@ DISCUSSION: If given a toolchain argument then `update` updates that toolchain, the same as `rustup toolchain install`. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml index 369cde2f13..78e398e1a0 100644 --- a/tests/suite/cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml @@ -5,18 +5,18 @@ stdout = """ Update Rust toolchains and rustup USAGE: - rustup[EXE] update [FLAGS] [toolchain]... - -FLAGS: - --force Force an update, even if some components are missing - --force-non-host Install toolchains that require an emulator. See https://github.com/rust- - lang/rustup/wiki/Non-host-toolchains - -h, --help Prints help information - --no-self-update Don't perform self update when running the `rustup update` command + rustup[EXE] update [OPTIONS] [toolchain]... ARGS: - ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help - toolchain` + ... Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + +OPTIONS: + --no-self-update Don't perform self update when running the `rustup update` command + --force Force an update, even if some components are missing + --force-non-host Install toolchains that require an emulator. See + https://github.com/rust-lang/rustup/wiki/Non-host-toolchains + -h, --help Print help information DISCUSSION: With no toolchain specified, the `update` command updates each of @@ -25,5 +25,6 @@ DISCUSSION: If given a toolchain argument then `update` updates that toolchain, the same as `rustup toolchain install`. + """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml index 06d6fa75a8..d22e3336a0 100644 --- a/tests/suite/cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml @@ -7,14 +7,13 @@ Display which binary will be run for a given command USAGE: rustup[EXE] which [OPTIONS] -FLAGS: - -h, --help Prints help information +ARGS: + OPTIONS: - --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see - `rustup help toolchain` + --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more + information see `rustup help toolchain` + -h, --help Print help information -ARGS: - """ stderr = "" From 795f9ef18024843c9745fe32136810477e1f8131 Mon Sep 17 00:00:00 2001 From: hi-rustin Date: Fri, 3 Mar 2023 23:17:03 +0800 Subject: [PATCH 5/6] Make match statements better --- src/cli/rustup_mode.rs | 152 ++++++++++++++++++++++++----------------- 1 file changed, 88 insertions(+), 64 deletions(-) diff --git a/src/cli/rustup_mode.rs b/src/cli/rustup_mode.rs index c6f8550d36..dcfebd0e34 100644 --- a/src/cli/rustup_mode.rs +++ b/src/cli/rustup_mode.rs @@ -128,72 +128,96 @@ pub fn main() -> Result { cfg.check_metadata_version()?; Ok(match matches.subcommand() { - Some(("dump-testament", _)) => common::dump_testament()?, - Some(("show", c)) => match c.subcommand() { - Some(("active-toolchain", m)) => handle_epipe(show_active_toolchain(cfg, m))?, - Some(("home", _)) => handle_epipe(show_rustup_home(cfg))?, - Some(("profile", _)) => handle_epipe(show_profile(cfg))?, - Some(("keys", _)) => handle_epipe(show_keys(cfg))?, - _ => handle_epipe(show(cfg, c))?, - }, - Some(("install", m)) => deprecated("toolchain install", cfg, m, update)?, - Some(("update", m)) => update(cfg, m)?, - Some(("check", _)) => check_updates(cfg)?, - Some(("uninstall", m)) => deprecated("toolchain uninstall", cfg, m, toolchain_remove)?, - Some(("default", m)) => default_(cfg, m)?, - Some(("toolchain", c)) => match c.subcommand() { - Some(("install", m)) => update(cfg, m)?, - Some(("list", m)) => handle_epipe(toolchain_list(cfg, m))?, - Some(("link", m)) => toolchain_link(cfg, m)?, - Some(("uninstall", m)) => toolchain_remove(cfg, m)?, - _ => unreachable!(), - }, - Some(("target", c)) => match c.subcommand() { - Some(("list", m)) => handle_epipe(target_list(cfg, m))?, - Some(("add", m)) => target_add(cfg, m)?, - Some(("remove", m)) => target_remove(cfg, m)?, - _ => unreachable!(), - }, - Some(("component", c)) => match c.subcommand() { - Some(("list", m)) => handle_epipe(component_list(cfg, m))?, - Some(("add", m)) => component_add(cfg, m)?, - Some(("remove", m)) => component_remove(cfg, m)?, - _ => unreachable!(), - }, - Some(("override", c)) => match c.subcommand() { - Some(("list", _)) => handle_epipe(common::list_overrides(cfg))?, - Some(("set", m)) => override_add(cfg, m)?, - Some(("unset", m)) => override_remove(cfg, m)?, - _ => unreachable!(), - }, - Some(("run", m)) => run(cfg, m)?, - Some(("which", m)) => which(cfg, m)?, - Some(("doc", m)) => doc(cfg, m)?, - Some(("man", m)) => man(cfg, m)?, - Some(("self", c)) => match c.subcommand() { - Some(("update", _)) => self_update::update(cfg)?, - Some(("uninstall", m)) => self_uninstall(m)?, - _ => unreachable!(), - }, - Some(("set", c)) => match c.subcommand() { - Some(("default-host", m)) => set_default_host_triple(cfg, m)?, - Some(("profile", m)) => set_profile(cfg, m)?, - Some(("auto-self-update", m)) => set_auto_self_update(cfg, m)?, + Some(s) => match s { + ("dump-testament", _) => common::dump_testament()?, + ("show", c) => match c.subcommand() { + Some(s) => match s { + ("active-toolchain", m) => handle_epipe(show_active_toolchain(cfg, m))?, + ("home", _) => handle_epipe(show_rustup_home(cfg))?, + ("profile", _) => handle_epipe(show_profile(cfg))?, + ("keys", _) => handle_epipe(show_keys(cfg))?, + _ => handle_epipe(show(cfg, c))?, + }, + None => handle_epipe(show(cfg, c))?, + }, + ("install", m) => deprecated("toolchain install", cfg, m, update)?, + ("update", m) => update(cfg, m)?, + ("check", _) => check_updates(cfg)?, + ("uninstall", m) => deprecated("toolchain uninstall", cfg, m, toolchain_remove)?, + ("default", m) => default_(cfg, m)?, + ("toolchain", c) => match c.subcommand() { + Some(s) => match s { + ("install", m) => update(cfg, m)?, + ("list", m) => handle_epipe(toolchain_list(cfg, m))?, + ("link", m) => toolchain_link(cfg, m)?, + ("uninstall", m) => toolchain_remove(cfg, m)?, + _ => unreachable!(), + }, + None => unreachable!(), + }, + ("target", c) => match c.subcommand() { + Some(s) => match s { + ("list", m) => handle_epipe(target_list(cfg, m))?, + ("add", m) => target_add(cfg, m)?, + ("remove", m) => target_remove(cfg, m)?, + _ => unreachable!(), + }, + None => unreachable!(), + }, + ("component", c) => match c.subcommand() { + Some(s) => match s { + ("list", m) => handle_epipe(component_list(cfg, m))?, + ("add", m) => component_add(cfg, m)?, + ("remove", m) => component_remove(cfg, m)?, + _ => unreachable!(), + }, + None => unreachable!(), + }, + ("override", c) => match c.subcommand() { + Some(s) => match s { + ("list", _) => handle_epipe(common::list_overrides(cfg))?, + ("set", m) => override_add(cfg, m)?, + ("unset", m) => override_remove(cfg, m)?, + _ => unreachable!(), + }, + None => unreachable!(), + }, + ("run", m) => run(cfg, m)?, + ("which", m) => which(cfg, m)?, + ("doc", m) => doc(cfg, m)?, + ("man", m) => man(cfg, m)?, + ("self", c) => match c.subcommand() { + Some(s) => match s { + ("update", _) => self_update::update(cfg)?, + ("uninstall", m) => self_uninstall(m)?, + _ => unreachable!(), + }, + None => unreachable!(), + }, + ("set", c) => match c.subcommand() { + Some(s) => match s { + ("default-host", m) => set_default_host_triple(cfg, m)?, + ("profile", m) => set_profile(cfg, m)?, + ("auto-self-update", m) => set_auto_self_update(cfg, m)?, + _ => unreachable!(), + }, + None => unreachable!(), + }, + ("completions", c) => { + if let Some(&shell) = c.get_one::("shell") { + output_completion_script( + shell, + c.get_one::("command") + .copied() + .unwrap_or(CompletionCommand::Rustup), + )? + } else { + unreachable!() + } + } _ => unreachable!(), }, - Some(("completions", c)) => { - if let Some(&shell) = c.get_one::("shell") { - (output_completion_script( - shell, - c.get_one::("command") - .copied() - .unwrap_or(CompletionCommand::Rustup), - ))? - } else { - unreachable!() - } - } - _ => unreachable!(), + None => unreachable!(), }) } From dae8b6a9e9a31a9584afc46d51fd0f36b5fabc3c Mon Sep 17 00:00:00 2001 From: hi-rustin Date: Sat, 4 Mar 2023 11:14:50 +0800 Subject: [PATCH 6/6] Remove extra new line after help --- rustup-init.sh | 1 - src/cli/rustup_mode.rs | 6 +++--- src/cli/setup_mode.rs | 2 +- .../cli-ui/rustup-init/rustup-init_help_flag_stdout.toml | 1 - .../cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml | 1 - .../rustup/rustup_completions_cmd_help_flag_stdout.toml | 1 - .../rustup_component_cmd_add_cmd_help_flag_stdout.toml | 1 - .../rustup_component_cmd_list_cmd_help_flag_stdout.toml | 1 - .../rustup_component_cmd_remove_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml | 1 - tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml | 1 - tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml | 1 - .../rustup_override_cmd_add_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml | 1 - .../rustup_override_cmd_list_cmd_help_flag_stdout.toml | 1 - .../rustup_override_cmd_remove_cmd_help_flag_stdout.toml | 1 - .../rustup_override_cmd_set_cmd_help_flag_stdout.toml | 1 - .../rustup_override_cmd_unset_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml | 1 - .../rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml | 1 - .../rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml | 1 - .../rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml | 1 - ...ustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml | 1 - .../rustup_set_cmd_default-host_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml | 1 - .../rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml | 1 - ...stup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml | 1 - .../rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml | 1 - .../rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml | 1 - .../rustup_show_cmd_profile_cmd_help_flag_stdout.toml | 1 - .../rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml | 1 - .../rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml | 1 - .../rustup_target_cmd_remove_cmd_help_flag_stdout.toml | 1 - .../rustup/rustup_toolchain_cmd_help_flag_stdout.toml | 1 - .../rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml | 1 - .../rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml | 1 - .../rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml | 1 - ...rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml | 1 - .../suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml | 1 - .../cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml | 1 - 48 files changed, 4 insertions(+), 50 deletions(-) diff --git a/rustup-init.sh b/rustup-init.sh index e9cfb2f9cf..9799046b1d 100755 --- a/rustup-init.sh +++ b/rustup-init.sh @@ -67,7 +67,6 @@ OPTIONS: -V, --version Print version information - EOF } diff --git a/src/cli/rustup_mode.rs b/src/cli/rustup_mode.rs index dcfebd0e34..ecb83806a4 100644 --- a/src/cli/rustup_mode.rs +++ b/src/cli/rustup_mode.rs @@ -71,11 +71,11 @@ pub fn main() -> Result { let matches = match cli().try_get_matches_from(process().args_os()) { Ok(matches) => Ok(matches), Err(err) if err.kind() == DisplayHelp => { - writeln!(process().stdout().lock(), "{err}")?; + write!(process().stdout().lock(), "{err}")?; return Ok(utils::ExitCode(0)); } Err(err) if err.kind() == DisplayVersion => { - writeln!(process().stdout().lock(), "{err}")?; + write!(process().stdout().lock(), "{err}")?; info!("This is the version for the rustup toolchain manager, not the rustc compiler."); fn rustc_version() -> std::result::Result> { @@ -107,7 +107,7 @@ pub fn main() -> Result { ] .contains(&err.kind()) { - writeln!(process().stdout().lock(), "{err}")?; + write!(process().stdout().lock(), "{err}")?; return Ok(utils::ExitCode(1)); } Err(err) diff --git a/src/cli/setup_mode.rs b/src/cli/setup_mode.rs index 25e4ecc258..6dd04ec2d4 100644 --- a/src/cli/setup_mode.rs +++ b/src/cli/setup_mode.rs @@ -105,7 +105,7 @@ pub fn main() -> Result { if e.kind() == clap::ErrorKind::DisplayHelp || e.kind() == clap::ErrorKind::DisplayVersion => { - writeln!(process().stdout().lock(), "{e}")?; + write!(process().stdout().lock(), "{e}")?; return Ok(utils::ExitCode(0)); } Err(e) => return Err(e.into()), diff --git a/tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml b/tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml index 1f67e80680..3aec80655d 100644 --- a/tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml @@ -44,6 +44,5 @@ OPTIONS: -V, --version Print version information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml b/tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml index 165e444002..706f9ec3c6 100644 --- a/tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml @@ -44,6 +44,5 @@ OPTIONS: -V, --version Print version information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml index 0be13627b2..793768eb60 100644 --- a/tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml @@ -9,6 +9,5 @@ USAGE: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml index c4f31ff299..309ed062fe 100644 --- a/tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml @@ -134,6 +134,5 @@ ${env:USERPROFILE}/Documents/WindowsPowerShell/Microsoft.PowerShell_profile.ps1 ZSH: $ rustup completions zsh cargo > ~/.zfunc/_cargo - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml index cb5f15b2b7..2dbd6b2192 100644 --- a/tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml @@ -15,6 +15,5 @@ OPTIONS: information see `rustup help toolchain` --target -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml index cb1516719a..221c90dcfa 100644 --- a/tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml @@ -12,6 +12,5 @@ OPTIONS: information see `rustup help toolchain` --installed List only installed components -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml index 3a84f7092c..917b72b041 100644 --- a/tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml @@ -15,6 +15,5 @@ OPTIONS: information see `rustup help toolchain` --target -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml index 1bb3bd154c..834081ef85 100644 --- a/tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml @@ -17,6 +17,5 @@ OPTIONS: DISCUSSION: Sets the default toolchain to the one specified. If the toolchain is not already installed then it is installed first. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml index 69e11f2c6a..ec12561e41 100644 --- a/tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml @@ -41,6 +41,5 @@ DISCUSSION: By default, it opens the documentation index. Use the various flags to open specific pieces of documentation. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml b/tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml index 05c1c7a523..65d9e2b2d5 100644 --- a/tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml @@ -44,6 +44,5 @@ DISCUSSION: If you are new to Rust consider running `rustup doc --book` to learn Rust. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml index 0964579015..03c6963a35 100644 --- a/tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml @@ -44,6 +44,5 @@ DISCUSSION: If you are new to Rust consider running `rustup doc --book` to learn Rust. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml index 962851a0b5..6ec2336396 100644 --- a/tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_man_cmd_help_flag_stdout.toml @@ -14,6 +14,5 @@ OPTIONS: --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help toolchain` -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_add_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_add_cmd_help_flag_stdout.toml index b5bd0f793f..d18211f0d7 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_add_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_add_cmd_help_flag_stdout.toml @@ -14,6 +14,5 @@ ARGS: OPTIONS: --path Path to the directory -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml index e99dedf9a9..08b2daecbd 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_help_flag_stdout.toml @@ -36,6 +36,5 @@ DISCUSSION: To see the active toolchain use `rustup show`. To remove the override and use the default toolchain again, `rustup override unset`. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_list_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_list_cmd_help_flag_stdout.toml index 7aeba37d02..8116d71751 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_list_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_list_cmd_help_flag_stdout.toml @@ -9,6 +9,5 @@ USAGE: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_remove_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_remove_cmd_help_flag_stdout.toml index 254a9e9b7d..2bc53d6720 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_remove_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_remove_cmd_help_flag_stdout.toml @@ -18,6 +18,5 @@ DISCUSSION: present, removes the override toolchain for all nonexistent directories. Otherwise, removes the override toolchain for the current directory. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_set_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_set_cmd_help_flag_stdout.toml index 3d669d3801..b1dc9680b5 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_set_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_set_cmd_help_flag_stdout.toml @@ -14,6 +14,5 @@ ARGS: OPTIONS: --path Path to the directory -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_override_cmd_unset_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_override_cmd_unset_cmd_help_flag_stdout.toml index 254a9e9b7d..2bc53d6720 100644 --- a/tests/suite/cli-ui/rustup/rustup_override_cmd_unset_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_override_cmd_unset_cmd_help_flag_stdout.toml @@ -18,6 +18,5 @@ DISCUSSION: present, removes the override toolchain for all nonexistent directories. Otherwise, removes the override toolchain for the current directory. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml index 20e1ee1a6b..9b38e69351 100644 --- a/tests/suite/cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_run_cmd_help_flag_stdout.toml @@ -30,6 +30,5 @@ DISCUSSION: $ cargo +nightly build $ rustup run nightly cargo build - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml index cf87e125d0..9c0ff9d9c3 100644 --- a/tests/suite/cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_self_cmd_help_flag_stdout.toml @@ -15,6 +15,5 @@ SUBCOMMANDS: uninstall Uninstall rustup. upgrade-data Upgrade the internal data format. help Print this message or the help of the given subcommand(s) - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml index add410f53a..28628dcd3a 100644 --- a/tests/suite/cli-ui/rustup/rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_self_cmd_uninstall_cmd_help_flag_stdout.toml @@ -10,6 +10,5 @@ USAGE: OPTIONS: -y -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml index e81a54107b..cbd215987a 100644 --- a/tests/suite/cli-ui/rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_self_cmd_update_cmd_help_flag_stdout.toml @@ -9,6 +9,5 @@ USAGE: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml index a9b9c02234..c06b9a03f9 100644 --- a/tests/suite/cli-ui/rustup/rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_self_cmd_upgrade-data _cmd_help_flag_stdout.toml @@ -9,6 +9,5 @@ USAGE: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml index 65508bbbc3..0bcce67e6f 100644 --- a/tests/suite/cli-ui/rustup/rustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_set_cmd_auto-self-update_cmd_help_flag_stdout.toml @@ -12,6 +12,5 @@ ARGS: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_set_cmd_default-host_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_set_cmd_default-host_cmd_help_flag_stdout.toml index 428953988f..3b5a9421ad 100644 --- a/tests/suite/cli-ui/rustup/rustup_set_cmd_default-host_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_set_cmd_default-host_cmd_help_flag_stdout.toml @@ -12,6 +12,5 @@ ARGS: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml index 20641c5c65..8522656129 100644 --- a/tests/suite/cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_set_cmd_help_flag_stdout.toml @@ -15,6 +15,5 @@ SUBCOMMANDS: profile The default components installed auto-self-update The rustup auto self update mode help Print this message or the help of the given subcommand(s) - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml index de26eb61fd..7013e21433 100644 --- a/tests/suite/cli-ui/rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_set_cmd_profile_cmd_help_flag_stdout.toml @@ -12,6 +12,5 @@ ARGS: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml index 1c8eca7e30..0bd8de5fe5 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_active-toolchain_cmd_help_flag_stdout.toml @@ -19,6 +19,5 @@ DISCUSSION: You should use `rustc --print sysroot` to get the sysroot, or `rustc --version` to get the toolchain version. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml index f99000e07c..e42f533792 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_help_flag_stdout.toml @@ -26,6 +26,5 @@ DISCUSSION: If there are multiple toolchains installed then all installed toolchains are listed as well. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml index d00738631e..a852be1033 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_home_cmd_help_flag_stdout.toml @@ -9,6 +9,5 @@ USAGE: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml index fd0d2dee99..104e7df922 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_keys_cmd_help_flag_stdout.toml @@ -9,6 +9,5 @@ USAGE: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_show_cmd_profile_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_show_cmd_profile_cmd_help_flag_stdout.toml index 49dbe0213b..32ec3e8347 100644 --- a/tests/suite/cli-ui/rustup/rustup_show_cmd_profile_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_show_cmd_profile_cmd_help_flag_stdout.toml @@ -9,6 +9,5 @@ USAGE: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml index fea2874f81..688b6f428e 100644 --- a/tests/suite/cli-ui/rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_target_cmd_add_cmd_help_flag_stdout.toml @@ -14,6 +14,5 @@ OPTIONS: --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help toolchain` -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml index 4948b5e0da..4872b92191 100644 --- a/tests/suite/cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_target_cmd_help_flag_stdout.toml @@ -15,6 +15,5 @@ SUBCOMMANDS: add Add a target to a Rust toolchain remove Remove a target from a Rust toolchain help Print this message or the help of the given subcommand(s) - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml index e8a424e714..07691abfc5 100644 --- a/tests/suite/cli-ui/rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_target_cmd_list_cmd_help_flag_stdout.toml @@ -12,6 +12,5 @@ OPTIONS: information see `rustup help toolchain` --installed List only installed targets -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_target_cmd_remove_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_target_cmd_remove_cmd_help_flag_stdout.toml index 6fecca9ccf..60abfefede 100644 --- a/tests/suite/cli-ui/rustup/rustup_target_cmd_remove_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_target_cmd_remove_cmd_help_flag_stdout.toml @@ -14,6 +14,5 @@ OPTIONS: --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help toolchain` -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_help_flag_stdout.toml index 6b81d52fba..7a9bc0faae 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_help_flag_stdout.toml @@ -58,6 +58,5 @@ DISCUSSION: rustup can also manage symlinked local toolchain builds, which are often used for developing Rust itself. For more information see `rustup toolchain help link`. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml index ae33af4b39..b3769d1b48 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_install_cmd_help_flag_stdout.toml @@ -23,6 +23,5 @@ OPTIONS: --force-non-host Install toolchains that require an emulator. See https://github.com/rust-lang/rustup/wiki/Non-host-toolchains -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml index 31d023c567..57bb36057b 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_link_cmd_help_flag_stdout.toml @@ -32,6 +32,5 @@ DISCUSSION: If you now compile a crate in the current directory, the custom toolchain 'latest-stage1' will be used. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml index 84db5f94e3..d2592440e9 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_list_cmd_help_flag_stdout.toml @@ -10,6 +10,5 @@ USAGE: OPTIONS: -v, --verbose Enable verbose output with toolchain information -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml index c88f683251..66a9f9b1d1 100644 --- a/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_toolchain_cmd_uninstall_cmd_help_flag_stdout.toml @@ -13,6 +13,5 @@ ARGS: OPTIONS: -h, --help Print help information - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml index 8122173b68..e60aced4fd 100644 --- a/tests/suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_up_cmd_help_flag_stdout.toml @@ -25,6 +25,5 @@ DISCUSSION: If given a toolchain argument then `update` updates that toolchain, the same as `rustup toolchain install`. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml index 166a9f5dba..40ffc935f7 100644 --- a/tests/suite/cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_update_cmd_help_flag_stdout.toml @@ -25,6 +25,5 @@ DISCUSSION: If given a toolchain argument then `update` updates that toolchain, the same as `rustup toolchain install`. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml index 78e398e1a0..2153d7ecbf 100644 --- a/tests/suite/cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_upgrade_cmd_help_flag_stdout.toml @@ -25,6 +25,5 @@ DISCUSSION: If given a toolchain argument then `update` updates that toolchain, the same as `rustup toolchain install`. - """ stderr = "" diff --git a/tests/suite/cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml b/tests/suite/cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml index d22e3336a0..97a4c96cd1 100644 --- a/tests/suite/cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml +++ b/tests/suite/cli-ui/rustup/rustup_which_cmd_help_flag_stdout.toml @@ -14,6 +14,5 @@ OPTIONS: --toolchain Toolchain name, such as 'stable', 'nightly', or '1.8.0'. For more information see `rustup help toolchain` -h, --help Print help information - """ stderr = ""