From 018255efe3e456aa8d712f68a714d5c6e010d03f Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Wed, 13 Nov 2019 10:27:21 +0300 Subject: Minor cleanup --- xtask/src/main.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'xtask/src/main.rs') diff --git a/xtask/src/main.rs b/xtask/src/main.rs index 16bddfb28..c46eaa407 100644 --- a/xtask/src/main.rs +++ b/xtask/src/main.rs @@ -19,7 +19,7 @@ use xtask::{ }; // Latest stable, feel free to send a PR if this lags behind. -const REQUIRED_RUST_VERSION: u32 = 38; +const REQUIRED_RUST_VERSION: u32 = 39; struct InstallOpt { client: Option, -- cgit v1.2.3 From 06754b78ac462d7b9fe8bd176ff7831772c0b1c8 Mon Sep 17 00:00:00 2001 From: kjeremy Date: Wed, 13 Nov 2019 11:19:07 -0500 Subject: Use autocfg to determine rust version --- xtask/src/main.rs | 35 ++++++++++------------------------- 1 file changed, 10 insertions(+), 25 deletions(-) (limited to 'xtask/src/main.rs') diff --git a/xtask/src/main.rs b/xtask/src/main.rs index c46eaa407..576ba003a 100644 --- a/xtask/src/main.rs +++ b/xtask/src/main.rs @@ -9,17 +9,18 @@ //! `.cargo/config`. mod help; +use autocfg; use core::fmt::Write; use core::str; use pico_args::Arguments; use std::{env, path::PathBuf}; use xtask::{ codegen::{self, Mode}, - install_format_hook, run, run_clippy, run_fuzzer, run_rustfmt, run_with_output, Cmd, Result, + install_format_hook, run, run_clippy, run_fuzzer, run_rustfmt, Cmd, Result, }; // Latest stable, feel free to send a PR if this lags behind. -const REQUIRED_RUST_VERSION: u32 = 39; +const REQUIRED_RUST_VERSION: (usize, usize) = (1, 39); struct InstallOpt { client: Option, @@ -226,20 +227,14 @@ fn install_client(ClientOpt::VsCode: ClientOpt) -> Result<()> { } fn install_server(opts: ServerOpt) -> Result<()> { - let mut old_rust = false; - if let Ok(output) = run_with_output("cargo --version", ".") { - if let Ok(stdout) = String::from_utf8(output.stdout) { - println!("{}", stdout); - if !check_version(&stdout, REQUIRED_RUST_VERSION) { - old_rust = true; - } - } - } + let ac = autocfg::AutoCfg::with_dir("target")?; + + let old_rust = !ac.probe_rustc_version(REQUIRED_RUST_VERSION.0, REQUIRED_RUST_VERSION.1); if old_rust { eprintln!( - "\nWARNING: at least rust 1.{}.0 is required to compile rust-analyzer\n", - REQUIRED_RUST_VERSION + "\nWARNING: at least rust {}.{}.0 is required to compile rust-analyzer\n", + REQUIRED_RUST_VERSION.0, REQUIRED_RUST_VERSION.1 ) } @@ -251,20 +246,10 @@ fn install_server(opts: ServerOpt) -> Result<()> { if res.is_err() && old_rust { eprintln!( - "\nWARNING: at least rust 1.{}.0 is required to compile rust-analyzer\n", - REQUIRED_RUST_VERSION + "\nWARNING: at least rust {}.{}.0 is required to compile rust-analyzer\n", + REQUIRED_RUST_VERSION.0, REQUIRED_RUST_VERSION.1 ) } res } - -fn check_version(version_output: &str, min_minor_version: u32) -> bool { - // Parse second the number out of - // cargo 1.39.0-beta (1c6ec66d5 2019-09-30) - let minor: Option = version_output.split('.').nth(1).and_then(|it| it.parse().ok()); - match minor { - None => true, - Some(minor) => minor >= min_minor_version, - } -} -- cgit v1.2.3 From 5e3c1c2b5f63e57f98a7d02f75a559d225341b1c Mon Sep 17 00:00:00 2001 From: Pascal Hertleif Date: Wed, 13 Nov 2019 20:44:39 +0100 Subject: Allow usage of CARGO_TARGET_DIR env var --- xtask/src/main.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'xtask/src/main.rs') diff --git a/xtask/src/main.rs b/xtask/src/main.rs index 576ba003a..84842b428 100644 --- a/xtask/src/main.rs +++ b/xtask/src/main.rs @@ -227,7 +227,8 @@ fn install_client(ClientOpt::VsCode: ClientOpt) -> Result<()> { } fn install_server(opts: ServerOpt) -> Result<()> { - let ac = autocfg::AutoCfg::with_dir("target")?; + let target_dir = env::var_os("CARGO_TARGET_DIR").unwrap_or_else(|| "target".into()); + let ac = autocfg::AutoCfg::with_dir(target_dir)?; let old_rust = !ac.probe_rustc_version(REQUIRED_RUST_VERSION.0, REQUIRED_RUST_VERSION.1); -- cgit v1.2.3 From 5075c77957e127be51e1b5271567f906abeb50c6 Mon Sep 17 00:00:00 2001 From: Pascal Hertleif Date: Wed, 13 Nov 2019 20:51:57 +0100 Subject: Use anyhow::Result in xtask, add contexts MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This builds on #2231 but was actually done before that. You see, the cause for #2231 was that I got this error message: Error: Error { kind: Io(Os { code: 2, kind: NotFound, message: "No such file or directory" }) } Just switching to `anyhow::Result` got me stack traces (when setting `RUST_LIB_BACKTRACE=1`) that at least showed stack backtrace: 0: std::backtrace::Backtrace::create 1: std::backtrace::Backtrace::capture 2: anyhow::error:: for anyhow::Error>::from 3: xtask::install_server 4: xtask::install 5: xtask::main 6: std::rt::lang_start::{{closure}} 7: std::panicking::try::do_call 8: __rust_maybe_catch_panic 9: std::rt::lang_start_internal 10: std::rt::lang_start 11: main With the added contexts (not at all exhaustive), the error became Error: install server Caused by: 0: build AutoCfg with target directory 1: No such file or directory (os error 2) Since anyhow is such a small thing (no new transitive dependencies!), and in general gives you `Result>` on steroids, I think this a nice small change. The only slightly annoying thing was to replace all the `Err(format!(…))?` calls (haven't even looked at whether we can make it support wrapping strings though), but the `bail!` macro is shorter anyway :) --- xtask/src/main.rs | 25 ++++++++++++++----------- 1 file changed, 14 insertions(+), 11 deletions(-) (limited to 'xtask/src/main.rs') diff --git a/xtask/src/main.rs b/xtask/src/main.rs index 84842b428..7eab1c949 100644 --- a/xtask/src/main.rs +++ b/xtask/src/main.rs @@ -9,6 +9,7 @@ //! `.cargo/config`. mod help; +use anyhow::Context; use autocfg; use core::fmt::Write; use core::str; @@ -114,21 +115,21 @@ fn handle_extra_flags(e: pico_args::Error) -> Result<()> { write!(&mut invalid_flags, "{}, ", flag)?; } let (invalid_flags, _) = invalid_flags.split_at(invalid_flags.len() - 2); - Err(format!("Invalid flags: {}", invalid_flags).into()) + anyhow::bail!("Invalid flags: {}", invalid_flags) } else { - Err(e.to_string().into()) + anyhow::bail!(e.to_string()) } } fn install(opts: InstallOpt) -> Result<()> { if cfg!(target_os = "macos") { - fix_path_for_mac()? + fix_path_for_mac().context("Fix path for mac")? } if let Some(server) = opts.server { - install_server(server)?; + install_server(server).context("install server")?; } if let Some(client) = opts.client { - install_client(client)?; + install_client(client).context("install client")?; } Ok(()) } @@ -140,7 +141,7 @@ fn fix_path_for_mac() -> Result<()> { const ROOT_DIR: &str = ""; let home_dir = match env::var("HOME") { Ok(home) => home, - Err(e) => Err(format!("Failed getting HOME from environment with error: {}.", e))?, + Err(e) => anyhow::bail!("Failed getting HOME from environment with error: {}.", e), }; [ROOT_DIR, &home_dir] @@ -154,12 +155,12 @@ fn fix_path_for_mac() -> Result<()> { if !vscode_path.is_empty() { let vars = match env::var_os("PATH") { Some(path) => path, - None => Err("Could not get PATH variable from env.")?, + None => anyhow::bail!("Could not get PATH variable from env."), }; let mut paths = env::split_paths(&vars).collect::>(); paths.append(&mut vscode_path); - let new_paths = env::join_paths(paths)?; + let new_paths = env::join_paths(paths).context("build env PATH")?; env::set_var("PATH", &new_paths); } @@ -198,7 +199,7 @@ fn install_client(ClientOpt::VsCode: ClientOpt) -> Result<()> { let code_binary = match code_binary { Some(it) => it, - None => Err("Can't execute `code --version`. Perhaps it is not in $PATH?")?, + None => anyhow::bail!("Can't execute `code --version`. Perhaps it is not in $PATH?"), }; Cmd { @@ -219,8 +220,10 @@ fn install_client(ClientOpt::VsCode: ClientOpt) -> Result<()> { .run_with_output()?; if !str::from_utf8(&output.stdout)?.contains("ra-lsp") { - Err("Could not install the Visual Studio Code extension. \ - Please make sure you have at least NodeJS 10.x installed and try again.")?; + anyhow::bail!( + "Could not install the Visual Studio Code extension. \ + Please make sure you have at least NodeJS 10.x installed and try again." + ); } Ok(()) -- cgit v1.2.3 From c65f42325fa9213ad79fcf12115213a923b651df Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Thu, 14 Nov 2019 11:11:32 +0300 Subject: Revert #2230 Looks like autocfg tries to do slightly more than we need (see #2231), so let's stick with minimal home-grown solution. --- xtask/src/main.rs | 36 +++++++++++++++++++++++++----------- 1 file changed, 25 insertions(+), 11 deletions(-) (limited to 'xtask/src/main.rs') diff --git a/xtask/src/main.rs b/xtask/src/main.rs index 7eab1c949..f14e6c8ae 100644 --- a/xtask/src/main.rs +++ b/xtask/src/main.rs @@ -10,18 +10,17 @@ mod help; use anyhow::Context; -use autocfg; use core::fmt::Write; use core::str; use pico_args::Arguments; use std::{env, path::PathBuf}; use xtask::{ codegen::{self, Mode}, - install_format_hook, run, run_clippy, run_fuzzer, run_rustfmt, Cmd, Result, + install_format_hook, run, run_clippy, run_fuzzer, run_rustfmt, run_with_output, Cmd, Result, }; // Latest stable, feel free to send a PR if this lags behind. -const REQUIRED_RUST_VERSION: (usize, usize) = (1, 39); +const REQUIRED_RUST_VERSION: u32 = 39; struct InstallOpt { client: Option, @@ -230,15 +229,20 @@ fn install_client(ClientOpt::VsCode: ClientOpt) -> Result<()> { } fn install_server(opts: ServerOpt) -> Result<()> { - let target_dir = env::var_os("CARGO_TARGET_DIR").unwrap_or_else(|| "target".into()); - let ac = autocfg::AutoCfg::with_dir(target_dir)?; - - let old_rust = !ac.probe_rustc_version(REQUIRED_RUST_VERSION.0, REQUIRED_RUST_VERSION.1); + let mut old_rust = false; + if let Ok(output) = run_with_output("cargo --version", ".") { + if let Ok(stdout) = String::from_utf8(output.stdout) { + println!("{}", stdout); + if !check_version(&stdout, REQUIRED_RUST_VERSION) { + old_rust = true; + } + } + } if old_rust { eprintln!( - "\nWARNING: at least rust {}.{}.0 is required to compile rust-analyzer\n", - REQUIRED_RUST_VERSION.0, REQUIRED_RUST_VERSION.1 + "\nWARNING: at least rust 1.{}.0 is required to compile rust-analyzer\n", + REQUIRED_RUST_VERSION, ) } @@ -250,10 +254,20 @@ fn install_server(opts: ServerOpt) -> Result<()> { if res.is_err() && old_rust { eprintln!( - "\nWARNING: at least rust {}.{}.0 is required to compile rust-analyzer\n", - REQUIRED_RUST_VERSION.0, REQUIRED_RUST_VERSION.1 + "\nWARNING: at least rust 1.{}.0 is required to compile rust-analyzer\n", + REQUIRED_RUST_VERSION, ) } res } + +fn check_version(version_output: &str, min_minor_version: u32) -> bool { + // Parse second the number out of + // cargo 1.39.0-beta (1c6ec66d5 2019-09-30) + let minor: Option = version_output.split('.').nth(1).and_then(|it| it.parse().ok()); + match minor { + None => true, + Some(minor) => minor >= min_minor_version, + } +} -- cgit v1.2.3