Skip to content

chore: Enable to build c2pa-python with a local c2pa-rs build - #277

Merged
tmathern merged 11 commits into
mainfrom
mathern/remove-obsolete-apis-local-build
Jun 8, 2026
Merged

chore: Enable to build c2pa-python with a local c2pa-rs build#277
tmathern merged 11 commits into
mainfrom
mathern/remove-obsolete-apis-local-build

Conversation

@tmathern

@tmathern tmathern commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Changes in this pull request

Enables debug builds based on checked out c2pa-rs/c2pa-c-ffi

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

@tmathern tmathern self-assigned this Jun 5, 2026
@tmathern tmathern changed the title chore: Enable to builde c2pa-python with a local c2pa-rs build chore: Enable to build c2pa-python with a local c2pa-rs build Jun 5, 2026
@tmathern
tmathern requested a review from ok-nick June 5, 2026 16:10
Comment thread scripts/build_local_artifacts.py

@ok-nick ok-nick left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does this script differ from the normal build workflow? It seems like it should be doing a lot of the the same work?

Comment thread scripts/build_local_artifacts.py Outdated
@tmathern

tmathern commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator Author

How does this script differ from the normal build workflow? It seems like it should be doing a lot of the the same work?

The assumptions differ. This one assumes you have a local c2pa-rs checkout next to this repo. The normal workflow takes/downloads a prebuilt released library (and does no rebuild). So it also asks you to have a local cargo setup (which you may not have if developing only Python code).

@tmathern
tmathern merged commit a33b0e5 into main Jun 8, 2026
23 checks passed
@tmathern
tmathern deleted the mathern/remove-obsolete-apis-local-build branch June 8, 2026 18:02
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.

2 participants