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

Upgrade Glow to 0.14 #4833

Closed
wants to merge 1 commit into from

Conversation

victoryforphil
Copy link

@victoryforphil victoryforphil commented Jul 16, 2024

Reason for Change

By updating glow to 0.14, this will pull a multiude of beneficial changes, such as Make debug message callback Send + Sync.
which is important when drawing three-d windows in a glow context.

Glow update: grovesNL/glow#296

Description of Changes

  • Bump glow to 0.14
  • No breaking changes notices from upgrade.

Verification

  • Ran ./scripts/check
  • Been running these changes manually on a in-use project
  • Closes: N/A (can make a ticket if requested)
  • I have followed the instructions in the PR template

Labels:

dependencies

@victoryforphil victoryforphil marked this pull request as ready for review July 16, 2024 20:29
@victoryforphil victoryforphil changed the title Updated Glow to 0.14 Bump Glow to 0.14 Jul 16, 2024
@victoryforphil victoryforphil changed the title Bump Glow to 0.14 Upgrade Glow to 0.14 Jul 16, 2024
@victoryforphil victoryforphil marked this pull request as draft July 17, 2024 16:41
@emilk
Copy link
Owner

emilk commented Jul 31, 2024

We've updated winit now:

Maybe it's time to revisit this PR?

@bircni bircni mentioned this pull request Aug 12, 2024
1 task
emilk added a commit that referenced this pull request Sep 9, 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`

```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]>
@victoryforphil victoryforphil deleted the vfp/glow_0_14 branch September 9, 2024 16:28
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants