Skip to content

simplicity sys: prepare simplicity-sys 0.8.0 release - #382

Merged
apoelstra merged 2 commits into
masterfrom
release/simplicity-sys-0.8.0
Sep 15, 2026
Merged

apoelstra merged 2 commits into
masterfrom
release/simplicity-sys-0.8.0

Conversation

@KyrylR

@KyrylR KyrylR commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

TL;DR: we are bumping because of braking change of bitcoin_hashes dep

Ran the vendor command with the following command:

nix shell nixpkgs#gnused nixpkgs#coreutils \                      
  --command bash ./vendor-simplicity.sh \
  ./depend /tmp/simplicity-vendor-check.PIsQKe/upstream

On this revision of Simplicity: BlockstreamResearch/simplicity@ba967b1

It is my first time doing such bump, so I am not entirely convinced that it is necessary, but AI reasoning (which is below) looks legit to me

Summary by GPT-6 Astra

Bump simplicity-sys to 0.8.0 ahead of the next Rust Simplicity release

The hash dependency update moved bitcoin_hashes from 0.14 to 1.x. Since simplicity-sys exposes these types in its public API, this is a breaking change and requires a version bump

Update the versioned C symbols, corresponding Rust bindings, and Cargo linking metadata to the 0.8 namespace. Update the workspace dependency requirements and lockfile accordingly. The vendored Simplicity revision and C implementation remain unchange

Verified that rerunning the vendoring script produces no Git diff

@KyrylR
KyrylR requested a review from apoelstra September 11, 2026 07:57
@KyrylR KyrylR self-assigned this Sep 11, 2026
@apoelstra

Copy link
Copy Markdown
Collaborator

Can you also add a pub use hashes; to src/lib.rs?

I didn't realize that we were exporting hashes types in our public API, but we are. So we should let users type simplicity_sys::hashes to get "the version of hashes used by simplicity-sys".

KyrylR added a commit that referenced this pull request Sep 14, 2026
Expose the hashes version used by the public midstate conversions and transaction environment API, as requested in PR #382 review. Prepared and verified with Codex.
@KyrylR
KyrylR force-pushed the release/simplicity-sys-0.8.0 branch from 1c3bfc2 to 731ae17 Compare September 14, 2026 09:55

@apoelstra apoelstra left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ACK 731ae17; successfully ran local tests

@KyrylR

KyrylR commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator Author

Tagged

@KyrylR

KyrylR commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator Author

@apoelstra I am not an owner of the sys package, so I cannot publish it

@apoelstra

Copy link
Copy Markdown
Collaborator

Deleted the tag, which should point to the merge commit (which does not exist yet).

@apoelstra

Copy link
Copy Markdown
Collaborator

Tagged and published.

@apoelstra
apoelstra merged commit c459a86 into master Sep 15, 2026
54 checks passed
@apoelstra
apoelstra deleted the release/simplicity-sys-0.8.0 branch September 15, 2026 17: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