Skip to content
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

splice: Move splice to experimental feature bit #6542

Merged

Conversation

ddustin
Copy link
Collaborator

@ddustin ddustin commented Aug 9, 2023

Fixes #6539

@ddustin ddustin requested a review from cdecker as a code owner August 9, 2023 16:39
@ddustin ddustin added this to the v23.08 milestone Aug 9, 2023
ddustin and others added 2 commits August 10, 2023 09:50
This was recommended by @t-bast: if the final spec commits to something
compatible, we can simply advertize and accept both features, but if it
does change in incompatible ways we won't cause problems for nodes
who implement the official spec.

(I split this, so first, we remove the OPT_SPLICE entirely, to make
sure we caught them all. --RR)

Suggested-by: @t-bast
Changelog-None
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@ddustin
Copy link
Collaborator Author

ddustin commented Aug 10, 2023

ACK fc98ed8

@rustyrussell rustyrussell merged commit 0281111 into ElementsProject:master Aug 10, 2023
39 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.

we should use feature bit 162 for splicing
2 participants