Skip to content

Conversation

@Galoretka
Copy link
Contributor

Replaced incorrect PSBT key name PSBT_IN_GLOBAL_XPUB with the canonical PSBT_GLOBAL_XPUB (per BIP 174). This corrects scope naming (GLOBAL vs per-input) and aligns with existing PSBT specifications.
Removed duplicated/garbled sentence and punctuation in the reference note explaining why compressed pubkeys are used instead of x-only. The note now clearly states the evenness-byte requirement and fingerprint rationale without repetition.

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK, nice fixups. Maybe optionally add this while here.

--- a/bip-0373.mediawiki
+++ b/bip-0373.mediawiki
@@ -9,13 +9,14 @@
   Type: Standards Track
   Created: 2024-06-04
   License: CC0-1.0
+  Requires: 32, 174, 327, 328, 370
 </pre>

@achow101
Copy link
Member

achow101 commented Dec 8, 2025

ACK d175b46

@Galoretka
Copy link
Contributor Author

+  Requires: 32, 174, 327, 328, 370

Oh, forgot about it, added

Copy link
Contributor

@murchandamus murchandamus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 0b3ab94

@murchandamus murchandamus merged commit 6ce21f4 into bitcoin:master Dec 9, 2025
4 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.

4 participants