Skip to content

v0.0.118 custom #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

v0.0.118 custom #2

wants to merge 4 commits into from

Conversation

orbitalturtle
Copy link
Owner

In this branch we just update to a specific hash of rust-lightning with the taggedhash tweaks we need for ldk-sample/lndk. We need the Cargo.toml changes to make sure we're using the exact same rust-lightning dependency everywhere, else we get version conflict errors in lndk.

We'll just use this temporary branch until ldk v119 comes out :)

We change the Bolt12Invoice struct to carry a tagged hash. Because
message_digest is then only used in one place, we can inline it in
the TaggedHash constructor.
Expose tag and merkle root fields in the TaggedHash struct.
orbitalturtle added a commit to lndk-org/ldk-sample that referenced this pull request Nov 15, 2023
As described in orbitalturtle/rust-lightning#2 we need
to temporarily bump rust-lightning dependencies to avoid dependency conflicts
in lndk.
orbitalturtle added a commit to lndk-org/ldk-sample that referenced this pull request Nov 15, 2023
As described in orbitalturtle/rust-lightning#2 we need
to temporarily bump rust-lightning dependencies to avoid dependency conflicts
in lndk.
orbitalturtle added a commit to lndk-org/ldk-sample that referenced this pull request Nov 22, 2023
As described in orbitalturtle/rust-lightning#2 we need
to temporarily bump rust-lightning dependencies to avoid dependency conflicts
in lndk.
orbitalturtle added a commit to lndk-org/ldk-sample that referenced this pull request Dec 6, 2023
As described in orbitalturtle/rust-lightning#2 we need
to temporarily bump rust-lightning dependencies to avoid dependency conflicts
in lndk.
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.

1 participant