-
Notifications
You must be signed in to change notification settings - Fork 995
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
[wgpu-hal] Upgrade to glutin
0.31.3, narrow libloading
to 0.8
#5649
Conversation
This comment was marked as duplicate.
This comment was marked as duplicate.
091fe5d
to
5720c19
Compare
@cwfitzgerald: It seems we need to pick an optional backend feature for |
They get automatically enabled, the issue is that iOS support was removed in v0.30. So, I think we should just skip building the example on iOS. |
What about this? Should we remove it or bump it as well? Line 154 in 77a83fb
|
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.
For tracking
@ErichDonGubler what's needed to finish this, port the I'd love to push this through to further reduce dependencies on |
@MarijnS95: Honestly, this just needs somebody to drive it to completion. I'm happy to cede this PR to you, if you have the bandwidth. |
@ErichDonGubler bandwidth is tight, but I've written and worked on my fair share of |
#5709 highlights that glutin drags in winit right now, making it a holdout for winit 0.27 |
This eliminates a ton of duplicate dependencies. Woot! Firefox is finally ready for incremental update of
libloading
, so we can make this change without any hassle. 😀Depends on #5504, to have no dependency onlibloading
0.7.