BIP Draft: rawtr() Output Script Descriptors - #2251
Conversation
|
Concept ACK |
There was a problem hiding this comment.
This was a very good read, thanks for submitting! It's coincise and easy to understand.
Looks good to me! Though I think someone who knows more about descriptors than I do should also take a look :)
EDIT: I just realized, you should change BIP380's Appendix B to point to this page.
| Title: rawtr() Output Script Descriptors | ||
| Authors: Jean Moreira <jeanpablo.jp@hotmail.com> | ||
| Status: Draft | ||
| Type: Specification |
There was a problem hiding this comment.
I think the type should be "Informational", similarly to other descriptor BIPs:
| BIP | Description | Type |
|---|---|---|
| 380 | Output Script Descriptors (general) | Informational |
| 381 | pk(), pkh(), multi() etc. | Informational |
| 382 | wpkh(), wsh() | Informational |
| 383 | sh() | Informational |
| 384 | combo() | Informational |
| 385 | raw(), addr() | Informational |
| 386 | tr() | Informational |
| 387 | multi_a(), sortedmulti_a() | Informational |
| 389 | Multipath descriptors | Informational |
| 390 | musig() (used inside rawtr()) | Informational |
There was a problem hiding this comment.
You are right, I just pushed the change.
Thanks.
54fea1a to
4af973f
Compare
|
@danielabrozzoni about the Appendix B, BIP 3 says only the editor assigns the number, and until then the BIP is referred to by name. I can add the row now pointing to |
This is a BIP draft for the
rawtr()script expression, following the style of BIPs 381-387. Bitcoin Core has implemented it since 24.0 and it has never had a BIP; sipa noted that on #2050 in March.While writing this I did some digging and found
rawtrin more places than I expected. BIP 390 allowsmusig()inside it, and two of the BIP 390 test vectors arerawtr()descriptors, so a published BIP already points at an expression no BIP defines. Outside Core it's a gap people route around: rust-miniscript printsrawtr_not_supported_yetwhere a descriptor should be, and bdk-ffi's tests citerawtras BIP-386.Mail list discussion: https://groups.google.com/g/bitcoindev/c/CCZN_qQ5C1s