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

Bump wgpu to 22.1 #111

Merged
merged 3 commits into from
Oct 23, 2024
Merged

Bump wgpu to 22.1 #111

merged 3 commits into from
Oct 23, 2024

Conversation

aftix
Copy link
Contributor

@aftix aftix commented Oct 22, 2024

Checklist

  • cargo clippy reports no issues
  • cargo doc reports no issues
  • cargo deny issues have been fixed or added to deny.toml
  • human-readable change descriptions added to the changelog under the "Unreleased" heading.
    • If the change does not affect the user (or is a process change), preface the change with "Internal:"
    • Add credit to yourself for each change: Added new functionality @githubname.

Description

Bumped wgpu to version 22.1. Consists of three commits:

  • bumping the dependency version, updating src/lib.rs slightly to the newer version
  • Fixing the examples with the new version of wgpu/winit
  • Adding skip rules to deny.toml

Related Issues

A few wgpu structs that this crate uses had fields added,
but they're ones where `Default::default` or `None` make sense,
so I've inserted them to get the build to work on the new version.

This is a breaking change for dependents relying on their own wgpu dependencies,
so I've bumped the version number and updated other dependencies that transitively
rely on wgpu.
Added the new wgpu structure fields in various places within the examples.
Also, since bumping wgpu meant bumping winit, the winit code in the examples
had become outdated (the `EventLoop::run` method was deprecated). I updated
the winit code to use `EventLoop::run_app`, which factors out the event
loop into a trait. I added a `struct App` for handling the event loop and a
`struct AppWindow` for handling setting up the window/gpu (needed for lifetime reasons)
to all examples.
The functionality is the same before and after the changes.
With the bump to wgpu 22.1, the hashbrown, syn, and bitflag
crates all trigger `cargo deny` due to multiple versions being found
in Cargo.lock. Different top-level dependencies are depending on different major
versions of these crates, and the top-level dependencies are all the most
recent version, so I don't think there's anything we can do to dedup these.
@Yatekii Yatekii merged commit 2edd348 into Yatekii:master Oct 23, 2024
5 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.

2 participants