-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Remove parking_lot dependency #20630
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
base: main
Are you sure you want to change the base?
Conversation
Your PR caused a change in the graphical output of an example or rendering test. This might be intentional, but it could also mean that something broke! If it's expected, please add the M-Deliberate-Rendering-Change label. If this change seems unrelated to your PR, you can consider updating your PR to target the latest main branch, either by rebasing or merging main into it. |
1 similar comment
Your PR caused a change in the graphical output of an example or rendering test. This might be intentional, but it could also mean that something broke! If it's expected, please add the M-Deliberate-Rendering-Change label. If this change seems unrelated to your PR, you can consider updating your PR to target the latest main branch, either by rebasing or merging main into it. |
Your PR caused a change in the graphical output of an example or rendering test. This might be intentional, but it could also mean that something broke! If it's expected, please add the M-Deliberate-Rendering-Change label. If this change seems unrelated to your PR, you can consider updating your PR to target the latest main branch, either by rebasing or merging main into it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for taking so long to get to this!
Objective
Spend less time compiling what is already in the standard library. Remove
parking_lot
from first party crate dependencies. Contribute to #18978. Supercedes #18996.Solution
Use
bevy_platform::sync
types wherever possible.BevyManifest::shared
directly returned a mapped guard, but that could be circumvented by just using a scope-like API instead.The crate is still in the dependency tree, transitively, through
wgpu
, but is now no longer required for any non-rendering crates.Testing
Ran tests locally.
Co-Authored-By: Zac Harrold [email protected]