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

M133 public #302

Merged
merged 7 commits into from
Feb 5, 2025
Merged

M133 public #302

merged 7 commits into from
Feb 5, 2025

Conversation

HinTak
Copy link
Collaborator

@HinTak HinTak commented Jan 8, 2025

Nothing interesting in this pull - just minimal routinely update to skia m133, and include #296 (breakage and compatibility with python 3.13.1).

@kyamagu : the mac ci has python 3.13.1 ; fedora also offers 3.13.1 from 3.13.0 about 3 weeks ago. There is probably a good reason to just push this out when people start to notice that pytthon 3.13.1 breaks. There are a number of good-to-go and work-in-progress pulls - I'd rather leave those until 134 (just to have a longer period between larger updates/additions).

See
python/cpython#128161
"Defining iterator in a separate class no longer works in 3.13"

We have iterator for SkTextBlob defined by SkTextBlob::Iter(textblob),
which is the c++/pybind11 equivalent of the same situation.
Following the suggestion:
python/cpython#128161 (comment)

Also see actions/runner-images#11241

Fixes kyamagu#295
…88ac03 to canvaskit/0.38.2-6918-gc17fe9bc15)
@HinTak HinTak mentioned this pull request Feb 4, 2025
@HinTak HinTak merged commit eacb758 into kyamagu:main Feb 5, 2025
18 checks passed
@HinTak HinTak deleted the m133-public branch February 5, 2025 17:47
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