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

Fix missing wgt re-export on wgpu for CopyExternalImageDestInfo #6962

Merged
merged 3 commits into from
Jan 20, 2025

Conversation

Wumpf
Copy link
Member

@Wumpf Wumpf commented Jan 20, 2025

Description
Fixes a missing export from wgpu

Testing
Use that type through wgpu's re-export (really, everything should imho)

Checklist

  • Run cargo fmt.
  • Run taplo format.
  • Run cargo clippy. If applicable, add:
    • --target wasm32-unknown-unknown
    • --target wasm32-unknown-emscripten
  • Run cargo xtask test to run tests.
  • Add change to CHANGELOG.md. See simple instructions inside file.

@Wumpf Wumpf added the PR: needs back-porting PR with a fix that needs to land on crates label Jan 20, 2025
@Wumpf Wumpf requested a review from a team as a code owner January 20, 2025 17:34
@Wumpf
Copy link
Member Author

Wumpf commented Jan 20, 2025

since wgt is not re-exported anywhere this is actually pretty bad. Forces users (me in this case ;)) to depend on wgt directly :/

Wumpf added a commit to rerun-io/rerun that referenced this pull request Jan 20, 2025
@Wumpf Wumpf enabled auto-merge (squash) January 20, 2025 17:51
@Wumpf Wumpf merged commit 426c1d3 into gfx-rs:trunk Jan 20, 2025
30 checks passed
@sagudev
Copy link
Contributor

sagudev commented Jan 20, 2025

You won't believe it, but this is something I also done in https://github.com/gfx-rs/wgpu/pull/6658/files#diff-69eb2a0461b18b418449129eaf12c15ee0a277bbaebb6860f2711de5fd6aa06a, I should split off minor changes more into separate PRs so they do not get blocked on big stuff.

cwfitzgerald pushed a commit that referenced this pull request Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: needs back-porting PR with a fix that needs to land on crates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants