Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: server hanging up on build script task #16616

Merged
merged 1 commit into from
Feb 20, 2024

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Feb 20, 2024

This should fix #16614, can't say for certain since it might be not 100% reproducible... We really need to replace the current workspace fetching logic, it is completely unreadable and incredibly difficult to follow. I don't really understand how the server even got to hang here honestly (I would expect it to loop re-fetching build scripts, but not hang).

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 20, 2024
@Veykril
Copy link
Member Author

Veykril commented Feb 20, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Feb 20, 2024

📌 Commit 9dee352 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Feb 20, 2024

⌛ Testing commit 9dee352 with merge 9f04957...

@bors
Copy link
Contributor

bors commented Feb 20, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 9f04957 to master...

@bors bors merged commit 9f04957 into rust-lang:master Feb 20, 2024
11 checks passed
@Veykril Veykril deleted the build-script-fix branch February 20, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rust-analyser hangs at "Fetching", and uses CPU at 100%
3 participants