Skip to content

Conversation

bircni
Copy link
Contributor

@bircni bircni commented Aug 12, 2024

Before making this PR, I did take notice of a similar PR, #4833, but as it appears to be abandoned, I decided to make this PR.

Missing
One of the checks doesn't pass as wgpu still uses glow 0.13.1

cargo deny --all-features --log-level error --target aarch64-apple-darwin check
  • I have followed the instructions in the PR template

@bircni
Copy link
Contributor Author

bircni commented Aug 12, 2024

glow is already on the main branch of wgpu so we just need to wait a bit
https://github.com/gfx-rs/wgpu/blob/7c917abf525b5c32c0b7345ec31788e316afa0cc/Cargo.toml#L162

@Wumpf
Copy link
Collaborator

Wumpf commented Sep 9, 2024

As mentioned here gfx-rs/wgpu#6240 (comment), this shouldn't be blocked on wgpu update. Just add a cargo deny exception here for the time being - the usecase of having both wgpu and glow backends of eframe enabled doesn't really exist, no?

@bircni
Copy link
Contributor Author

bircni commented Sep 9, 2024

@Wumpf you are right gonna add this asap

@bircni bircni marked this pull request as ready for review September 9, 2024 11:33
@emilk emilk added the egui_glow Relates to running egui_glow on native label Sep 9, 2024
@emilk emilk added the dependencies Pull requests that update a dependency file label Sep 9, 2024
@emilk emilk changed the title Bump glow to 0.14 Update glow to 0.14 Sep 9, 2024
@emilk emilk merged commit 1c293d4 into emilk:master Sep 9, 2024
21 checks passed
@bircni bircni deleted the patch01 branch September 9, 2024 13:16
hacknus pushed a commit to hacknus/egui that referenced this pull request Oct 30, 2024
Before making this PR, I did take notice of a similar PR,
emilk#4833, but as it appears to be
abandoned, I decided to make this PR.

**Missing**
One of the checks doesn't pass as wgpu still uses glow `0.13.1`

```shell
cargo deny --all-features --log-level error --target aarch64-apple-darwin check
```

* [x] I have followed the instructions in the PR template

---------

Co-authored-by: Emil Ernerfeldt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file egui_glow Relates to running egui_glow on native

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants