Repository navigation
Breaking change: upgrade to c2pa-rs 0.92.0 (minor version bump) - #270
Merged
Merged
Conversation
Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
Replace deleted constructors and settings APIs, adapt opaque streams, and migrate embeddable signing to context signers. Fix source-build artifacts and Windows toolchain setup. Co-authored-by: Copilot App <[email protected]>
Keep APIs removed by the c2pa-rs main migration removed while incorporating upstream changes. Co-authored-by: Copilot App <[email protected]>
Pin the compatibility CI checkout to the released c2pa-rs RC tag. Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
tmathern
reviewed
Nov 11, 2026
Bump the C++ minor version to 0.27.0, document the migration, and validate both source and prebuilt dependencies in CI. Co-authored-by: Copilot App <[email protected]>
Show the supported context-based reader instead of code using the removed load_settings API. Co-authored-by: Copilot App <[email protected]>
Keep removed APIs in migration guidance and correct descriptions of stream-based hashing. Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
Introduce CppStream with a virtual destructor and shared opaque-handle ownership, preserving c_stream access on all wrappers. Co-authored-by: Copilot App <[email protected]>
scouten-adobe
commented
Nov 11, 2026
| - name: Checkout repository | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Checkout c2pa-rs (0.92.0) |
Contributor
Author
There was a problem hiding this comment.
I'll remove this once the 0.92.0 binaries are available.
Co-authored-by: Copilot App <[email protected]>
Distinguish c2pa::C2paStream from the opaque global C handle and document the compatibility boundary. Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
scouten-adobe
marked this pull request as ready for review
November 11, 2026 22:01
Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
tmathern
approved these changes
Nov 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Breaking change: upgrade to c2pa-rs 0.92.0
Update the native SDK dependency to the official
c2pa-v0.92.0release and bump the C++ project version from0.26.19to0.27.0. The release tag resolves to c2pa-rs commit83eba34d1a0b6be4d2484fc5d7b4cf11c030b694.Migration
c2pa::load_settingshas been removed. Configure settings using aContext; context-freeReaderandBuilderconstructors now use default settings rather than thread-local settings.Builder::data_hashed_placeholderandBuilder::sign_data_hashed_embeddablehave been removed. Attach a signer to theContextand useplaceholder,set_data_hash_exclusions,update_hash_from_stream, andsign_embeddable. The new workflow does not accept precomputed DataHash JSON.c2pa::C2paStreambase. The global C API type::C2paStreamis opaque; use the wrapper'sc_streammember for C API calls.c2pa_free.Builder::format_embeddableremains available viac2pa_builder_compose_manifest.See the release notes and migration documentation for details. CI uses the official 0.92.0 prebuilt release archives; it no longer checks out or builds c2pa-rs source.
Preliminary validation