From 1d1eea217d5b674e6cad77704f912a3a76505d70 Mon Sep 17 00:00:00 2001 From: Emil Lauridsen Date: Fri, 10 Jan 2020 22:10:26 +0100 Subject: Slightly more robust cargo watcher root search --- crates/ra_lsp_server/src/world.rs | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) (limited to 'crates/ra_lsp_server') diff --git a/crates/ra_lsp_server/src/world.rs b/crates/ra_lsp_server/src/world.rs index 121ddfd1f..36b5242d1 100644 --- a/crates/ra_lsp_server/src/world.rs +++ b/crates/ra_lsp_server/src/world.rs @@ -74,7 +74,7 @@ impl WorldState { lru_capacity: Option, exclude_globs: &[Glob], watch: Watch, - options: Options, + mut options: Options, feature_flags: FeatureFlags, ) -> WorldState { let mut change = AnalysisChange::new(); @@ -132,8 +132,20 @@ impl WorldState { change.set_crate_graph(crate_graph); // FIXME: Figure out the multi-workspace situation - let check_watcher = - CheckWatcher::new(&options.cargo_watch, folder_roots.first().cloned().unwrap()); + let check_watcher = { + let first_workspace = workspaces.first().unwrap(); + let cargo_project_root = match first_workspace { + ProjectWorkspace::Cargo { cargo, .. } => cargo.workspace_root.clone(), + ProjectWorkspace::Json { .. } => { + log::warn!( + "Cargo check watching only supported for cargo workspaces, disabling" + ); + options.cargo_watch.enable = false; + PathBuf::new() + } + }; + CheckWatcher::new(&options.cargo_watch, cargo_project_root) + }; let mut analysis_host = AnalysisHost::new(lru_capacity, feature_flags); analysis_host.apply_change(change); -- cgit v1.2.3 From d6da18e99d2fb4e67e3bc7503059282b5e14bd13 Mon Sep 17 00:00:00 2001 From: Emil Lauridsen Date: Fri, 10 Jan 2020 22:41:52 +0100 Subject: Address nit --- crates/ra_lsp_server/src/world.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'crates/ra_lsp_server') diff --git a/crates/ra_lsp_server/src/world.rs b/crates/ra_lsp_server/src/world.rs index 36b5242d1..cea18937f 100644 --- a/crates/ra_lsp_server/src/world.rs +++ b/crates/ra_lsp_server/src/world.rs @@ -135,7 +135,7 @@ impl WorldState { let check_watcher = { let first_workspace = workspaces.first().unwrap(); let cargo_project_root = match first_workspace { - ProjectWorkspace::Cargo { cargo, .. } => cargo.workspace_root.clone(), + ProjectWorkspace::Cargo { cargo, .. } => cargo.workspace_root().to_path_buf(), ProjectWorkspace::Json { .. } => { log::warn!( "Cargo check watching only supported for cargo workspaces, disabling" -- cgit v1.2.3 From 480c44918c73922b134b670cbd33014b8fcfc883 Mon Sep 17 00:00:00 2001 From: Emil Lauridsen Date: Sat, 11 Jan 2020 21:32:40 +0100 Subject: Disable cargo checking in workspaces with no cargo projects --- crates/ra_lsp_server/src/world.rs | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) (limited to 'crates/ra_lsp_server') diff --git a/crates/ra_lsp_server/src/world.rs b/crates/ra_lsp_server/src/world.rs index cea18937f..a52bd2633 100644 --- a/crates/ra_lsp_server/src/world.rs +++ b/crates/ra_lsp_server/src/world.rs @@ -132,20 +132,20 @@ impl WorldState { change.set_crate_graph(crate_graph); // FIXME: Figure out the multi-workspace situation - let check_watcher = { - let first_workspace = workspaces.first().unwrap(); - let cargo_project_root = match first_workspace { - ProjectWorkspace::Cargo { cargo, .. } => cargo.workspace_root().to_path_buf(), - ProjectWorkspace::Json { .. } => { - log::warn!( - "Cargo check watching only supported for cargo workspaces, disabling" - ); - options.cargo_watch.enable = false; - PathBuf::new() - } - }; - CheckWatcher::new(&options.cargo_watch, cargo_project_root) - }; + let check_watcher = workspaces + .iter() + .find_map(|w| match w { + ProjectWorkspace::Cargo { cargo, .. } => Some(cargo), + ProjectWorkspace::Json { .. } => None, + }) + .map(|cargo| { + let cargo_project_root = cargo.workspace_root().to_path_buf(); + CheckWatcher::new(&options.cargo_watch, cargo_project_root) + }) + .unwrap_or_else(|| { + log::warn!("Cargo check watching only supported for cargo workspaces, disabling"); + CheckWatcher::dummy() + }); let mut analysis_host = AnalysisHost::new(lru_capacity, feature_flags); analysis_host.apply_change(change); -- cgit v1.2.3 From 8e778f9842123e1f688a2632d99e439821801bd2 Mon Sep 17 00:00:00 2001 From: Emil Lauridsen Date: Sat, 11 Jan 2020 21:37:01 +0100 Subject: Clean up straggling mut --- crates/ra_lsp_server/src/world.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'crates/ra_lsp_server') diff --git a/crates/ra_lsp_server/src/world.rs b/crates/ra_lsp_server/src/world.rs index a52bd2633..c0175c726 100644 --- a/crates/ra_lsp_server/src/world.rs +++ b/crates/ra_lsp_server/src/world.rs @@ -74,7 +74,7 @@ impl WorldState { lru_capacity: Option, exclude_globs: &[Glob], watch: Watch, - mut options: Options, + options: Options, feature_flags: FeatureFlags, ) -> WorldState { let mut change = AnalysisChange::new(); -- cgit v1.2.3