Skip to content

Commit

Permalink
fix: server hanging up on build script task
Browse files Browse the repository at this point in the history
  • Loading branch information
Veykril committed Feb 20, 2024
1 parent 26a16c4 commit 9dee352
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 21 deletions.
20 changes: 4 additions & 16 deletions crates/rust-analyzer/src/global_state.rs
Original file line number Diff line number Diff line change
Expand Up @@ -301,19 +301,12 @@ impl GlobalState {
if let Some(path) = vfs_path.as_path() {
let path = path.to_path_buf();
if reload::should_refresh_for_change(&path, file.kind()) {
workspace_structure_change = Some((
path.clone(),
false,
AsRef::<std::path::Path>::as_ref(&path).ends_with("build.rs"),
));
workspace_structure_change = Some((path.clone(), false));
}
if file.is_created_or_deleted() {
has_structure_changes = true;
workspace_structure_change = Some((
path,
self.crate_graph_file_dependencies.contains(vfs_path),
false,
));
workspace_structure_change =
Some((path, self.crate_graph_file_dependencies.contains(vfs_path)));
} else if path.extension() == Some("rs".as_ref()) {
modified_rust_files.push(file.file_id);
}
Expand Down Expand Up @@ -365,16 +358,11 @@ impl GlobalState {
// FIXME: ideally we should only trigger a workspace fetch for non-library changes
// but something's going wrong with the source root business when we add a new local
// crate see https://github.com/rust-lang/rust-analyzer/issues/13029
if let Some((path, force_crate_graph_reload, build_scripts_touched)) =
workspace_structure_change
{
if let Some((path, force_crate_graph_reload)) = workspace_structure_change {
self.fetch_workspaces_queue.request_op(
format!("workspace vfs file change: {path}"),
force_crate_graph_reload,
);
if build_scripts_touched {
self.fetch_build_data_queue.request_op(format!("build.rs changed: {path}"), ());
}
}
}

Expand Down
1 change: 1 addition & 0 deletions crates/rust-analyzer/src/lsp/utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ impl GlobalState {
let token = lsp_types::ProgressToken::String(
cancel_token.unwrap_or_else(|| format!("rustAnalyzer/{title}")),
);
tracing::debug!(?token, ?state, "report_progress {message:?}");
let work_done_progress = match state {
Progress::Begin => {
self.send_request::<lsp_types::request::WorkDoneProgressCreate>(
Expand Down
7 changes: 2 additions & 5 deletions crates/rust-analyzer/src/reload.rs
Original file line number Diff line number Diff line change
Expand Up @@ -411,10 +411,7 @@ impl GlobalState {
if *force_reload_crate_graph {
self.recreate_crate_graph(cause);
}
if self.build_deps_changed && self.config.run_build_scripts() {
self.build_deps_changed = false;
self.fetch_build_data_queue.request_op("build_deps_changed".to_owned(), ());
}

// Current build scripts do not match the version of the active
// workspace, so there's nothing for us to update.
return;
Expand All @@ -424,7 +421,7 @@ impl GlobalState {

// Here, we completely changed the workspace (Cargo.toml edit), so
// we don't care about build-script results, they are stale.
// FIXME: can we abort the build scripts here?
// FIXME: can we abort the build scripts here if they are already running?
self.workspaces = Arc::new(workspaces);

if self.config.run_build_scripts() {
Expand Down

0 comments on commit 9dee352

Please sign in to comment.