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 glow to 0.14 #4952

Merged
merged 4 commits into from
Sep 9, 2024
Merged

Update glow to 0.14 #4952

merged 4 commits into from
Sep 9, 2024

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
deny.toml Outdated Show resolved Hide resolved
@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