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

update to latest wgpu-core #318

Merged
merged 3 commits into from
Nov 17, 2023
Merged

update to latest wgpu-core #318

merged 3 commits into from
Nov 17, 2023

Conversation

radgeRayden
Copy link
Contributor

@radgeRayden radgeRayden commented Nov 11, 2023

Fixes version conflicts, updates the naga url to point to the main wgpu repo, some changes that arise from the bump to raw-window-handle 0.6.0. I am a Rust noob so please let me know if anything is amiss or bad style.

@radgeRayden
Copy link
Contributor Author

I realize now that I wasn't seeing checks for non-Linux code paths which introduced some errors I didn't catch. Will fix it all tomorrow.

And a couple of formatting issues.
@rajveermalviya
Copy link
Collaborator

rajveermalviya commented Nov 16, 2023

(Sorry for the late merge, somehow this got left out.)

After this is merged wgpu-native's trunk will target wgpu trunk, so any future changes would not be released until next major wgpu release, though we could backport some of them if possible to the v0.18 branch and make a dot release.

@almarklein
Copy link
Collaborator

wgpu-core 0.18.1 released yesterday, should we update to that in a new pr? Perhaps create a v0.18 branch?

@rajveermalviya
Copy link
Collaborator

wgpu-core 0.18.1 released yesterday, should we update to that in a new pr?

Created #319, let's merge that one first then this one.

@rajveermalviya
Copy link
Collaborator

rajveermalviya commented Nov 17, 2023

Merging! Thanks!

@rajveermalviya rajveermalviya enabled auto-merge (squash) November 17, 2023 10:38
@rajveermalviya rajveermalviya merged commit 9105402 into gfx-rs:trunk Nov 17, 2023
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants