Skip to content

Conversation

@DJMcNab
Copy link
Member

@DJMcNab DJMcNab commented Sep 9, 2025

This is a backport of #126, as discussed in #linebender > raw_resource_handle crate extraction from Peniko @ 💬

This fails Cargo Semver Checks because of obi1kenobi/cargo-semver-checks#638. Therefore, I have manually checked that this works in Xilem and Cosmic Text.

Those tests were performed using:

[patch.crates-io]
peniko = { git = "https://github.com/linebender/peniko", rev = "d86ffcd5e5a3853a0a4f57774649e6075424bd6d" }

This is the more permanent equivalent of #107, and will mean that 0.4.x, 0.3.x and 0.5.x are all using the same Font/Blob types.

@DJMcNab DJMcNab changed the base branch from main to v0.4.x September 9, 2025 16:19
DJMcNab added a commit to DJMcNab/cosmic-text that referenced this pull request Sep 9, 2025
This is not to be merged, but is instead being used to validate linebender/peniko#129
by running your CI

Note that it's likely that Cosmic text will want to replace your
Peniko usage with Linebender Resource Handle instead, once Peniko 0.4.1 is out
Copy link
Contributor

@nicoburns nicoburns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM subject to backwards compat checking

@DJMcNab DJMcNab force-pushed the linebender_resource_handle_backport branch from d86ffcd to 715cf10 Compare September 10, 2025 09:42
@DJMcNab
Copy link
Member Author

DJMcNab commented Sep 10, 2025

As discussed in #126, I've tweaked the CHANGELOG, therefore I've requested a new review.

I don't plan to merge this before Office Hours tomorrow, to get final consensus confirmed.

@DJMcNab DJMcNab requested a review from nicoburns September 10, 2025 09:59
@DJMcNab DJMcNab merged commit a369f05 into linebender:v0.4.x Sep 12, 2025
15 checks passed
@DJMcNab DJMcNab deleted the linebender_resource_handle_backport branch September 12, 2025 13:30
DJMcNab added a commit that referenced this pull request Sep 15, 2025
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