Skip to content

Add BIN-2025-0003, OP_PAIRCOMMIT#17

Closed
Ademan wants to merge 0 commit intobitcoin-inquisition:masterfrom
Ademan:paircommit
Closed

Add BIN-2025-0003, OP_PAIRCOMMIT#17
Ademan wants to merge 0 commit intobitcoin-inquisition:masterfrom
Ademan:paircommit

Conversation

@Ademan
Copy link
Copy Markdown

@Ademan Ademan commented Nov 22, 2025

Since PAIRCOMMIT has been assigned BIP-442 does it make the most sense to just point the binana to the (currently not yet merged) BIP?

This is an alternative approach to #8

I made an effort to follow existing conventions but I wasn't sure about a few things:

  1. Self assignment of the BIP number->BIN number, is this OK or should I use BIN-2025-0003, or even steal OP_PAIRCOMMIT #8 's BIN-2024-0006 ? (BIP-119 which I copied referenced an older version scheme, is that still applicable here?)
  2. PAIRCOMMIT has been assigned a BIP number but hasn't been merged yet, so this is pointing to something outside of the official BIPs repo, is this OK?
  3. Wasn't sure what date to use for the Revision field, just used the current date, should it be the date of the referenced commit instead?

Finally, my ultimate goal is to add PAIRCOMMIT as inquisition heretical deployments, it appeared to me that this is a prerequisite to getting that upstreamed into bitcoin-inquisition/bitcoin , right?

@ajtowns
Copy link
Copy Markdown
Collaborator

ajtowns commented Dec 4, 2025

BIN16-118 and BIN16-119's numbers are historical exceptions, and shouldn't be used as examples. Taking over BIN24-6 from #8 would make sense if the proposers of that are happy with it; if it's a divergent proposal, though, then it should get it's own number, I think BIN25-3 would be next?

Pointing to any reasonable permalink seems fine, particularly if a commit hash is included. If the permalink turns out not to be permanent, can always update it later, using the commit hash to ensure consistency.

Revision date should match the referenced document, but as long as it's not in the future, probably doesn't matter too much.

Inquisition heretical activations require a BINANA number, yeah.

@moonsettler
Copy link
Copy Markdown

Feel free to take over, or tell me what to do with #8! I could update the PR to the latest against the BIP repo, but that is still under review and might receive feedback that results in a change.

@Ademan Ademan changed the title Add BIN-2025-0442, OP_PAIRCOMMIT Add BIN-2025-0003, OP_PAIRCOMMIT Dec 7, 2025
@Ademan
Copy link
Copy Markdown
Author

Ademan commented Dec 7, 2025

I renamed this binana to BIN25-3 just to dodge any uncertainty about superceding #8.

I also updated the date to match the BIP date. The PAIRCOMMIT BIP hasn't been moving too fast lately but I'll make an effort to keep the pointer in this binana up to date.

Thanks for the feedback and guidance!

@moonsettler moonsettler mentioned this pull request Dec 7, 2025
ajtowns added a commit that referenced this pull request Apr 24, 2026
This introduces a BIN for the `OP_TEMPLATEHASH` proposal as drafted in
bitcoin/bips#1974 and implemented in
bitcoin-inquisition/bitcoin#100.

The proposal dates from late 2025 but BIN-26-1 was chosen in place of
BIN-25-4 since the BIP number 446 was assigned on February 6, 2026.
Since the revision date [should
match](#17 (comment))
the referenced document, i went with 26.

Since the BIP is not merged yet, we link to the PR instead of a commit
hash on the author's branch, which would be much more volatile due to
force pushes in addressing review comments. Once the BIP is merged, the
revision will be updated to a commit hash.
@ajtowns ajtowns closed this Apr 24, 2026
@ajtowns
Copy link
Copy Markdown
Collaborator

ajtowns commented Apr 24, 2026

Ugh, sorry about that. Tried doing the rebase myself to see if the new workflow was operative and messed up, and now github won't let me fix it. You should be able to rebase yourself and reopen the PR.

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