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

Feature/skia m99 #165

Merged
merged 14 commits into from
Feb 2, 2022
Merged

Feature/skia m99 #165

merged 14 commits into from
Feb 2, 2022

Conversation

chrfalch
Copy link
Contributor

@chrfalch chrfalch commented Feb 2, 2022

After meeting with Google we decided to try to stay on the latest Skia release at all times. In addition we have a few outstanding issues with running on the Apple iOS M1 Simulator in Xcode that we want to address. We also don't emit bitcode for the device builds on iOS - this is also addressed.

  • Upgraded Skia to version chrome/m99
  • Upgraded Jsi API fixing breaking changes
    • JsiSkCanvas::drawPatch: SkCanvas::drawPatch needs a default blend mode (@wcandillon What is the default blend mode?)
    • JsiSkRuntimeEffect: SkRuntimeEffect.uniforms has a few small API changes
  • Updated build tools and build configuration to be able to build multiple targets for same arc
  • Added notes in Readme files
  • Updated workflows to be manually triggered

fixes #161
fixes #47
fixes #35

@chrfalch chrfalch marked this pull request as ready for review February 2, 2022 10:04
@wcandillon wcandillon self-requested a review February 2, 2022 10:23
@wcandillon
Copy link
Contributor

@ChrisJamesC this enables RN Skia to run on arm64 simulator

@wcandillon wcandillon merged commit d6e449e into main Feb 2, 2022
@wcandillon wcandillon deleted the feature/skia-m99 branch April 7, 2022 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants