Script: SCRIPT_VERIFY_SIGPUSHONLY sits unused (except tests) #26113

issue instagibbs openend this issue on September 16, 2022
  1. instagibbs commented at 5:51 pm on September 16, 2022: member

    Turns out this flag was added for BIP62, which was withdrawn a long time ago. It’s only used in tests, not in policy or consensus otherwise.

    If there’s no plans on picking the BIP back up, better to remove?

    h/t @ajtowns

  2. ajtowns commented at 10:00 am on September 17, 2022: member
    Picking it back up was part of #15482 / the great consensus cleanup, and I don’t think there were any objections to that part of it. Not sure how much difference it would make to keep it vs removing it and re-adding it later though.
  3. fanquake added the label Brainstorming on Sep 19, 2022
  4. glozow commented at 10:23 am on October 6, 2022: member
    Maybe it should be policy? :shrug:
  5. instagibbs commented at 1:28 pm on October 6, 2022: member
    Great Policy Cleanup incoming?
  6. instagibbs commented at 1:32 pm on October 6, 2022: member
    More seriously, the pushonly check is already done in policy via IsStandardTx
  7. hebasto deleted a comment on Oct 7, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-21 09:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me