Skip to content

fix: compile raw-gles example again after #6150 #6174

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

Conversation

ErichDonGubler
Copy link
Member

@ErichDonGubler ErichDonGubler commented Aug 28, 2024

Connections

I think this was regressed by #6150?

Description

See above.

Testing

It compiles now!

Checklist

  • Run cargo fmt.

  • 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.

    Probably not necessary? Might want to add on to the glutin CHANGELOG entry. 🤔

@ErichDonGubler ErichDonGubler added the backend: gles Issues with GLES or WebGL label Aug 28, 2024
@ErichDonGubler ErichDonGubler self-assigned this Aug 28, 2024
@ErichDonGubler
Copy link
Member Author

@MarijnS95 ☝🏻

@MarijnS95
Copy link
Contributor

Doesn't your CI catch this?

Seems like I squashed this into the wrong PR (https://github.com/gfx-rs/wgpu/pull/6152/files#diff-ac2020a9d8bd724543c3bf9ff45b699d1ae9b15a9f6d77c38a604c7ff73cb716R19) as I was rebasing this from a Windows machine, which isn't supported in #6150 yet but only starting with #6152.

@teoxoy teoxoy closed this in #6176 Aug 28, 2024
@ErichDonGubler ErichDonGubler deleted the erichdongubler/push-mwmukqrqrmmz branch August 28, 2024 12:09
@ErichDonGubler
Copy link
Member Author

@MarijnS95: I'm not sure why our CI doesn't catch it. @cwfitzgerald, is there a reason CI doesn't at least try to compile examples?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend: gles Issues with GLES or WebGL
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants