chore: bump skrifa + harfrust

[akx]

This makes the default build's dep graph unify on a single read-fonts 0.39.x + single version of skrifa.

Before

~/b/cosmic-text (main) $ git rev-parse HEAD
c24886c2471e5606587c46090cd25dbbf209186b
~/b/cosmic-text (main) $ cargo build -q
~/b/cosmic-text (main) $ cargo tree -d -e normal
read-fonts v0.37.0
├── harfrust v0.5.2
│   └── cosmic-text v0.19.0 (/Users/akx/build/cosmic-text)
└── skrifa v0.40.0
    └── cosmic-text v0.19.0 (/Users/akx/build/cosmic-text)
read-fonts v0.39.2
└── skrifa v0.42.1
    └── swash v0.2.9
        └── cosmic-text v0.19.0 (/Users/akx/build/cosmic-text)
skrifa v0.40.0 (*)
skrifa v0.42.1 (*)

After

~/b/cosmic-text (skrifa-042) $ cargo build -q
~/b/cosmic-text (skrifa-042) $ cargo tree -d -e normal
warning: nothing to print.

harfrust's shape_with_plan had become private in 0.8.0, so change that use.

---

• I have disclosed use of any AI generated code in my commit messages.
• I understand these changes in full and will be able to respond to review comments.
• My change is accurately described in the commit message.
• My contribution is tested and working as described.
  • cargo test passes, and an upstream app using this code seems to work as before.
• I have read the Developer Certificate of Origin and certify my contribution under its conditions.

[akx]

Hmm... It built fine on my machine 😅

[hojjatabdollahi]

Make sure to compile it with no-std feature too.

[norepro]

Not sure if this should be separate or a change to this PR but, if you can update harfrust to 0.11.0 or newer, it would also fix pop-os/cosmic-files#1895 (once it takes whatever release of cosmic-text has this change).

[akx]

Not sure if this should be separate or a change to this PR but, if you can update harfrust to 0.11.0 or newer

Ah yeah, that wasn't an option when I opened this PR, since that would've pulled in multiple versions of font-types, but since Skrifa has since updated, let's see... googlefonts/fontations@cd4aaef

EDIT: Not right now, because that will require cooperation from Swash as well, or otherwise we're again pulling in two different skrifas and read-fontses. I've created an upstream PR: dfrg/swash#131

[oscargus]

Maybe add a range for skrifa (and harfrust)?

It is indeed a bit annoying to have to synchronize bumped version over lots of crates...

[HigherOrderLogic]

I also prefer a range here, which would help rust-windowing/winit#4640. Currently, cargo deny is complaining about 2 different skrifa and harfrust version from cosmic-text and sctk-adwaita itself.

[akx]

@norepro The Swash PR was merged, so I added another commit to further bump to harfrust 0.12.0, skrifa 0.44.0, swash 0.2.10.

@oscargus @HigherOrderLogic What kind of range would you suggest? 🤔

[oscargus]

0.40-0.44

https://github.com/PolyMeilex/sctk-adwaita/blob/1bddc2264a71ae3897903ec8a7d7b13b834ff1b1/Cargo.toml#L30
https://github.com/dfrg/swash/blob/7773843df0d63cd468db61a29c152b5e7a99d4ab/Cargo.toml#L37

[oscargus]

Or really as large as possible, but a bit tedious to test too many... For winit, I think the main thing is that the upper range is not higher than 0.44. (Now 0.45 is released...)

[HigherOrderLogic]

For winit, I think the main thing is that the upper range is not higher than 0.44

I dont think Winit uses skrifa directly, but it's from sctk-adwaita instead. Ideally, the skrifa range in cosmic-text should match what's there in sctk-adwaita, but I dont know how hard it is to keep them up to date with each other.

[oscargus]

Yeah, it is more that winit runs a cargo deny job that will complain when cargo resolves to different versions. (Although I think they allowed multiple skrifa versions.)

[oscargus]

Which I realize that you know @HigherOrderLogic ... Just wanted to motivate that statement.