Skip to content

Upgrade Glow to 0.14#4833

Closed
victoryforphil wants to merge 1 commit intoemilk:masterfrom
victoryforphil:vfp/glow_0_14
Closed

Upgrade Glow to 0.14#4833
victoryforphil wants to merge 1 commit intoemilk:masterfrom
victoryforphil:vfp/glow_0_14

Conversation

@victoryforphil
Copy link
Copy Markdown

@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
Copy Markdown
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 <emil.ernerfeldt@gmail.com>
@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 <emil.ernerfeldt@gmail.com>
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