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 to rusttype 0.8.3 #1411

Merged
merged 4 commits into from
Apr 14, 2021

Conversation

mitchmindtree
Copy link
Contributor

@mitchmindtree mitchmindtree commented Apr 7, 2021

Also removes some unnecessary buffering of positioned glyphs.

Edit: For context on the following, this PR originally tried updating to 0.9.2.

Landing this is blocked on a patch that exists on rusttype master but has not yet been released:

https://gitlab.redox-os.org/redox-os/rusttype/-/merge_requests/158

Unfortunately, rusttype is currently lacking a maintainer and as a result it's hard to say when/if we'll be able to land this:

https://gitlab.redox-os.org/redox-os/rusttype/-/issues/148

@mitchmindtree mitchmindtree changed the title [WIP] Update to latest version of rusttype Update to latest version of rusttype (blocked on finding a rusttype maintainer) Apr 7, 2021
@mitchmindtree mitchmindtree changed the title Update to latest version of rusttype (blocked on finding a rusttype maintainer) Update to rusttype 0.8.3 (blocked on finding a rusttype maintainer) Apr 14, 2021
@mitchmindtree mitchmindtree changed the title Update to rusttype 0.8.3 (blocked on finding a rusttype maintainer) Update to rusttype 0.8.3 Apr 14, 2021
Also removes some unnecessary buffering of positioned glyphs.

The patch required fixes an otherwise inconsolable lifetime issue
related to the positioned glyphs yielded by a font's layout method.
@mitchmindtree
Copy link
Contributor Author

It looks like we can still get away with using 0.8.3 - it seems that whatever upstream breakage I was running into might have been yanked from crates.io in the meantime.

I think we're probably better off sticking to 0.8.3 for now, at least until either rusttype gets a new maintainer or someone is motivated enough to switch over to the ab_glyph spiritual-successor crate.

The vulkano CI breakage is unrelated (see #1409).

@mitchmindtree mitchmindtree merged commit 8531889 into PistonDevelopers:master Apr 14, 2021
@mitchmindtree mitchmindtree deleted the rusttype-0.9.2 branch April 14, 2021 14:48
mitchmindtree added a commit to mitchmindtree/conrod that referenced this pull request Apr 15, 2021
Changes include:

- Update to rusttype 0.8.3 with more efficient text rendering PistonDevelopers#1411.
  This slightly affects how the `Text` render primitive is handled.
  Folks with custom conrod backends may want to take a peak at this PR
  to see how to update.
- Update to wgpu 0.7 and winit 0.24 in `conrod_wgpu`. PistonDevelopers#1413
- Reduce deps by updating to `num` 0.3. PistonDevelopers#1395

NOTE: `conrod_vulkano` will be excluded from this version update due to
`vk-sys` publishing a breaking change not too long ago. See PistonDevelopers#1409.
mitchmindtree added a commit to mitchmindtree/conrod that referenced this pull request Apr 15, 2021
Changes include:

- Update to rusttype 0.8.3 with more efficient text rendering PistonDevelopers#1411.
  This slightly affects how the `Text` render primitive is handled.
  Folks with custom conrod backends may want to take a peak at this PR
  to see how to update.
- Update to wgpu 0.7 and winit 0.24 in `conrod_wgpu`. PistonDevelopers#1413
- Reduce deps by updating to `num` 0.3. PistonDevelopers#1395

NOTE: `conrod_vulkano` will be excluded from this version update due to
`vk-sys` publishing a breaking change not too long ago. See PistonDevelopers#1409.
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.

1 participant