-
Notifications
You must be signed in to change notification settings - Fork 165
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
Remove the windows-sys dependency #591
Comments
See here for an example of how this might be done. |
I've seen Amanieu/parking_lot#374 which seems to contain the main rationale for this, and I'm surprised that there seems to be an absence of any kind of numbers describing how much compile time this saves, or even how much filesystem cache space or network traffic it saves or anything else. I'm not eager to take on new kinds of complexity to solve an abstract problem. |
I plan on elucidating all of this in a blog post soon, parking_lot was simply the first crate I chose while iterating on a generation tool since it is foundational and thus has an outsize ecosystem effect, and was also just low hanging fruit that I was fairly confident would be merged. |
Closing for now. |
The
windows-sys
crate has gone through quite a bit of version churn, see microsoft/windows-rs#1720. Therefore, some of the foundational crates in the ecosystem have been manually creating Win32 bindings for what they need.rustix
may want to migrate in this direction as well.The text was updated successfully, but these errors were encountered: