Skip to content

README: sync BIP-93 author list with its header - #2281

Open
kriss39 wants to merge 1 commit into
bitcoin:masterfrom
kriss39:fix/readme-bip93-authors
Open

README: sync BIP-93 author list with its header#2281
kriss39 wants to merge 1 commit into
bitcoin:masterfrom
kriss39:fix/readme-bip93-authors

Conversation

@kriss39

@kriss39 kriss39 commented Sep 11, 2026

Copy link
Copy Markdown

BIP-93 credits its first two authors as a single entry sharing one address:

  Authors: Leon Olsson Curr and Pearlwort Sneed <pearlwort@wpsoftware.net>
           Andrew Poelstra <andrew.poelstra@gmail.com>

The table row split that entry on a comma, so it reads as three authors with three addresses. buildtable.pl keeps the whole name ahead of each address and joins entries with a comma, so it renders the row as Leon Olsson Curr and Pearlwort Sneed, Andrew Poelstra.

Running scripts/buildtable.pl and diffing against the committed table, this was the only row where the two disagreed on content. With this change the two match everywhere except the BIP number allocated rows for 40, 41 and 63, which have no files for the script to read and are maintained by hand.

scripts/link-format-chk.sh, scripts/buildtable.pl and typos all still pass.

BIP-93 credits its first two authors as one entry sharing one address:

  Authors: Leon Olsson Curr and Pearlwort Sneed <pearlwort@wpsoftware.net>
           Andrew Poelstra <andrew.poelstra@gmail.com>

The table row split that entry on a comma, which reads as three authors with
three addresses. buildtable.pl keeps the name ahead of each address intact and
joins entries with a comma, so it renders the row the other way.

With this, the committed table matches the generated one everywhere except the
three "BIP number allocated" rows for 40, 41 and 63, which have no files for
the script to read and are maintained by hand.
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