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

feat: upgrade thorvg v1.0-pre8 #244

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

theashraf
Copy link
Member

@theashraf theashraf commented Oct 11, 2024

Changes

  1. Feature Flags for thorvg Versions

    • Added two feature flags: thorvg-v1 and thorvg-v0.
    • These allow us to maintain compatibility with thorvg v0.x.x, as binaries for Linux, Windows, and macOS from the Conan registry currently only support v0.
    • thorvg-v1 is a pre-release version and is not yet stable or available on Conan.
  2. Default Configuration

    • By default, the thorvg-v0 flag is enabled for the FFI, unless explicitly specified otherwise.
  3. Build Target Differences

    • WebAssembly (WASM), Android, and Apple builds use thorvg-v1.
    • Native dependencies (Linux/Windows/macOS) use `thorvg-v0

WASM build -> https://github.com/LottieFiles/dotlottie-rs/actions/runs/12251496921
Apple build -> https://github.com/LottieFiles/dotlottie-rs/actions/runs/12251496917
Android build -> https://github.com/LottieFiles/dotlottie-rs/actions/runs/12251496922

Copy link

changeset-bot bot commented Oct 11, 2024

⚠️ No Changeset found

Latest commit: ad237de

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@theashraf
Copy link
Member Author

Breaking changes in the new version
image

@theashraf
Copy link
Member Author

update thorvg version to 1.0-pre1 in Conan registry
conan-io/conan-center-index#25592

@theashraf theashraf changed the title chore: Update thorvg v1.0-pre1 chore: update thorvg v1.0-pre7 Dec 9, 2024
@theashraf theashraf force-pushed the chore/thorvg-1.0-pre1 branch 2 times, most recently from aa24c96 to bdac0e2 Compare December 10, 2024 06:55
@theashraf theashraf changed the title chore: update thorvg v1.0-pre7 feat: upgrade thorvg v1.0-pre7 Dec 10, 2024
@theashraf
Copy link
Member Author

thorvg v1 rendering regression issue -> thorvg/thorvg#3039

- Update the build workflow to include pull request labeled events for triggering builds.
- Refactor the concurrency group to include the target or label name in the group name.
- Update the build-apple job to include the 'build:apple' label as a trigger condition.
- Update the build-android job to include the 'build:android' label as a trigger condition.
- Update the build-wasm job to include the 'build:wasm' label as a trigger condition.

chore: Update Makefile

- Add the '-Dengines=sw' flag to the meson setup to enable the software rendering engine.

chore: Update dotlottie-ffi dependencies

- Update the version of dotlottie-ffi to '0.1.31' in the Cargo.toml file.

chore: Update dotlottie-ffi Cargo.wasm.toml

- Remove the 'codegen-units' and 'strip' options from the Cargo.wasm.toml file.

chore: Update dotlottie-rs dependencies

- Update the version of thiserror to '2.0.4' in the Cargo.toml file.

chore: Update dotlottie-rs build.rs

- Update the bindgen version to '0.71.0' in the build.rs file.

chore: Update dotlottie-rs lottie_renderer/mod.rs

- Update the serde_json version to '1.0.133' in the mod.rs file.

chore: Update dotlottie-rs lottie_renderer/renderer.rs

- Update the clear method signature in the renderer.rs file to include separate flags for clearing paints and buffers.

chore: Update dotlottie-rs lottie_renderer/thorvg.rs

- Update the clear method signature in the thorvg.rs file to include separate flags for clearing paints and buffers.

feat: Add thorvg-v1 feature to dotlottie-rs

- Add the 'thorvg-v1' feature to the dotlottie-rs Cargo.toml file.

fix: Update DotLottieRuntime and DotLottiePlayerContainer

- Update the new method signatures in the DotLottieRuntime and DotLottiePlayerContainer structs to include conditional compilation based on the 'thorvg-v0' and 'thorvg-v1' features.

fix: Update DotLottiePlayer

- Update the new method signature in the DotLottiePlayer struct to include conditional compilation based on the 'thorvg-v0' and 'thorvg-v1' features.

fix: Update LottieRendererImpl

- Update the render method in the LottieRendererImpl struct to include conditional compilation based on the 'thorvg-v1' feature.

fix: Update TvgRenderer

- Update the clear method in the TvgRenderer struct to include conditional compilation based on the 'thorvg-v1' feature.

docs: Update README.md

- Update the README.md file to reflect the recent changes.
Update the clippy command in the Makefile to use the --no-default-features flag and add the thorvg-v1 feature. Also, update the build workflow to remove the --all-features flag when running cargo clippy for dotlottie-rs and dotlottie-ffi.
@theashraf theashraf force-pushed the chore/thorvg-1.0-pre1 branch from 88bf479 to 2158fd6 Compare December 13, 2024 15:12
@theashraf theashraf changed the title feat: upgrade thorvg v1.0-pre7 feat: upgrade thorvg v1.0-pre8 Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant