From f89722f3571bcdcd7f5e9b6d22d4d8d82dffc332 Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Sat, 27 Jun 2020 02:00:08 +0200 Subject: Better exe probing --- crates/ra_toolchain/src/lib.rs | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) (limited to 'crates/ra_toolchain/src/lib.rs') diff --git a/crates/ra_toolchain/src/lib.rs b/crates/ra_toolchain/src/lib.rs index 9532fb255..9916e52c4 100644 --- a/crates/ra_toolchain/src/lib.rs +++ b/crates/ra_toolchain/src/lib.rs @@ -46,22 +46,23 @@ fn get_path_for_executable(executable_name: &'static str) -> PathBuf { path.push(".cargo"); path.push("bin"); path.push(executable_name); - if path.is_file() { + if let Some(path) = probe(path) { return path; } } + executable_name.into() } fn lookup_in_path(exec: &str) -> bool { let paths = env::var_os("PATH").unwrap_or_default(); - let mut candidates = env::split_paths(&paths).flat_map(|path| { - let candidate = path.join(&exec); - let with_exe = match env::consts::EXE_EXTENSION { - "" => None, - it => Some(candidate.with_extension(it)), - }; - iter::once(candidate).chain(with_exe) - }); - candidates.any(|it| it.is_file()) + env::split_paths(&paths).map(|path| path.join(exec)).find_map(probe).is_some() +} + +fn probe(path: PathBuf) -> Option { + let with_extension = match env::consts::EXE_EXTENSION { + "" => None, + it => Some(path.with_extension(it)), + }; + iter::once(path).chain(with_extension).find(|it| it.is_file()) } -- cgit v1.2.3