Skip to content

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
@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