Skip to content

Conversation

@spacebear21
Copy link
Collaborator

With the changes from Uniffi-Dart/uniffi-dart#99, the native libraries can be compiled automagically in a dart build hook, a feature added very recently in Dart 3.10 which solves a lot of packaging headaches.

Pull Request Checklist

Please confirm the following before requesting review:

@coveralls
Copy link
Collaborator

coveralls commented Nov 26, 2025

Pull Request Test Coverage Report for Build 19910130353

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 83.539%

Totals Coverage Status
Change from base Build 19909591402: 0.0%
Covered Lines: 8988
Relevant Lines: 10759

💛 - Coveralls

@spacebear21 spacebear21 force-pushed the ffi-dart-native-assets branch 2 times, most recently from 8abfcec to be117a8 Compare November 26, 2025 22:10
@spacebear21 spacebear21 force-pushed the ffi-dart-native-assets branch 5 times, most recently from ee4e79d to 5a49eb5 Compare December 3, 2025 19:13
@spacebear21 spacebear21 marked this pull request as ready for review December 3, 2025 19:20
@spacebear21 spacebear21 requested a review from benalleng December 3, 2025 19:20
@spacebear21
Copy link
Collaborator Author

@benalleng looks like the cargo-mutants Diff CI job is failing due to a modification to uniffi.toml that it's failing to parse. It seemed nontrivial to change the diff script to ignore toml files, but I think it's safe to ignore that failing check for this particular PR.

@spacebear21 spacebear21 force-pushed the ffi-dart-native-assets branch 2 times, most recently from 59dc734 to 3592336 Compare December 3, 2025 19:32
This changes the bitcoin-ffi revision accordingly to one that supports
0.30.
With the changes from
Uniffi-Dart/uniffi-dart#99, the native libraries
can be compiled automagically in a dart build hook.
uniffi::generate_scaffolding! is only necessary when using UDL
(https://mozilla.github.io/uniffi-rs/latest/tutorial/Rust_scaffolding.html).

We use proc macros so we can skip build.rs entirely!
@spacebear21 spacebear21 force-pushed the ffi-dart-native-assets branch from 1a87042 to 52a4538 Compare December 3, 2025 21:56
Copy link
Collaborator

@benalleng benalleng left a comment

Choose a reason for hiding this comment

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

uTACK on 52a4538 thanks for remembering to switch over from my bitcoin-ffi fork!

@spacebear21 spacebear21 merged commit 3f057e1 into payjoin:master Dec 4, 2025
16 checks passed
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.

3 participants