Skip to content

Conversation

RCasatta
Copy link
Collaborator

No description provided.

@apoelstra
Copy link
Member

Heh, this crate was last touched 8 months ago. I will take some time today to try to get it into working shape.

@apoelstra
Copy link
Member

apoelstra commented Aug 26, 2025

If you approve #98 and rebase this on that it should be good to go.

RCasatta added a commit that referenced this pull request Aug 27, 2025
…py lints

a4ec6fb ci: fix doclinks (Andrew Poelstra)
8dd5e75 clippy: enable a bunch of casting and safety lints (Andrew Poelstra)
92fc444 ci: copy workflow and contrib/ scripts from rust-elements (Andrew Poelstra)
c62e83b clippy: introduce nightly-version and fix all clippy issues with it (Andrew Poelstra)

Pull request description:

  Should get CI passing. Then we can rebase #97.

ACKs for top commit:
  RCasatta:
    utACK a4ec6fb

Tree-SHA512: 078f28230466f8bac28bb2cc3071852151cc2d1f1e4b7921e2edd2cf91f668d3109130db21290afd4170f40192bcb3130cde69b3ba90cbc0a15fff3f1e8c3f5c
@apoelstra
Copy link
Member

Now you need to update the included Cargo-latest.lock file :).

@RCasatta RCasatta force-pushed the bump_elements branch 2 times, most recently from ce7048b to 2f88724 Compare August 27, 2025 13:12
Signed-off-by: Riccardo Casatta <[email protected]>
@apoelstra
Copy link
Member

Looks like there's an incompatibility between elements 0.25 (included by simplicity) and elements 0.26.

Updating simplicity is going to be a bunch of work (though we should do it). Let's go to rust-elements and release a 0.25.3 which re-exports all the types from 0.26.0, since they're almost all the same.

@apoelstra
Copy link
Member

Ok, nice, we can do the semver trick (this is the first time I've tried it and wasn't sure if it would work). But we need to release an elements 0.26.1 first.

apoelstra added a commit to ElementsProject/rust-elements that referenced this pull request Aug 28, 2025
05a24ad bump version to 0.26.1 (Andrew Poelstra)
2b89d5f encode: implement Encodable and Decodable for Vec<T> (Andrew Poelstra)
45a6938 export pset::error::PsetHash (Andrew Poelstra)

Pull request description:

  I want to do the dtolnay "semver trick" where I release 0.25.3 which re-exports most of the types and traits from 0.26. This will make migration easier for users since they could use elements 0.25 and 0.26 in the same crate and have compatible types. In particular ElementsProject/elements-miniscript#97 would compile without needing to update `simplicity` and `elements` at the same time.
  
  However, when doing this I identified some gaps in the API which make this harder than it should be. See the commit messages for detailed justifications.
  
  Also release 0.26.1 with these changes.


ACKs for top commit:
  RCasatta:
    utACK 05a24ad


Tree-SHA512: 447d09950586da98a1d8a13fe8dc1ce69761d2a3152b9ea243264ec874f04bc1219d42a562e0470246c266980e77d2a46d807c78ed5ca9bbcf8f4055af3ea4c6
@apoelstra
Copy link
Member

Can you add a [patch] section to Cargo.toml which replaces elements 0.25 with ElementsProject/rust-elements#251 (which unfortunately we can't merge and release)?

Then after that, I guess we need to add a commit which updates the simplicity dep and removes the patch.

But at least we can do it across multiple commits now.

@RCasatta
Copy link
Collaborator Author

RCasatta commented Sep 1, 2025

in the first commit I downgraded elements to 0.25.3 and use patch in f79d75e

then I upgraded simplicity and just added a dummy env to "fix" compilation, not sure how the env should be handled correctly

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