BIP-175: fix links #1862

pull strmfos wants to merge 1 commits into bitcoin:master from strmfos:master changing 1 files +1 −1
  1. strmfos commented at 10:42 am on May 30, 2025: contributor
    PR updates outdated references to the deprecated commerceblock/copay repository and replaces them with the actively maintained bitpay/wallet repository. The change ensures that dependencies and links point to the correct, maintained source.
  2. Migrate references from deprecated commerceblock/copay to active bitpay/wallet 291ed0f006
  3. murchandamus changes_requested
  4. murchandamus commented at 6:03 pm on May 30, 2025: contributor
    Are you sure you copied the right link? As far as I can tell, the suggested link goes to a pull request that does not have anything to do with a reference implementation of BIP 175.
  5. strmfos commented at 6:55 am on May 31, 2025: contributor

    https://github.com/BitLox/copay

    Hey! This is a maintained fork of the original Copay wallet that supports the BIP-175 Pay to Contract Protocol implementation. The original commerceblock/copay repository is no longer available, but this fork preserves the required functionality for hierarchical deterministic (HD) wallet address generation with BIP32/BIP44 support.

    Is this an appropriate replacement link, or do you have a better alternative you would recommend?

  6. jonatack commented at 3:23 pm on June 1, 2025: member
    As @murchandamus points out, the pull request link doesn’t point to the code implementation. @strmfos Would you please update the link to one that does, or to the relevant section of the codebase, if self-contained? A repository that may contain the functionality somewhere is a bit too large.
  7. strmfos commented at 9:37 am on June 2, 2025: contributor
    I found a specific implementation link for BIP-175: https://github.com/commerceblock/pay-to-contract-lib This repo contains the Hash to Partial Derivation Path Mapping function in contract.js file - exactly as described in the BIP-175 spec. Is this suitable as replacement reference?
  8. jonatack commented at 5:39 pm on June 2, 2025: member

    @strmfos That link looks redundant to the one in the next line; implementation-wise it only has that contract.js file.

    This is a maintained fork of the original Copay wallet that supports the BIP-175 Pay to Contract Protocol

    I might be missing something, but that repo references several BIPs, yet I didn’t see a reference there to BIP175.

    As BIP175’s status is Rejected, perhaps remove line 247 containing the broken link?

  9. strmfos commented at 5:46 pm on June 2, 2025: contributor

    I really reviewed a lot to find a proper replacement for the link, but I’ve shared everything I found.

    So it might be reasonable to remove line 247 for now—if that works for you, I’ll go ahead and do it.

  10. jonatack commented at 8:20 pm on June 2, 2025: member
    Done in #1864 and credited to you (thanks!), so closing here.
  11. jonatack closed this on Jun 2, 2025


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-06-18 19:10 UTC

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