Should release-process.md still mention verify.sh? #7595

issue JeremyRand opened this issue on February 25, 2016
  1. JeremyRand commented at 1:10 AM on February 25, 2016: contributor

    As far as I can tell, verify.sh hasn't been updated for new Bitcoin releases in quite some time. #2504 states that verify.sh isn't supposed to guess what version to download anyway; I'm not sure if that's the current situation. However, release-process.md lists verify.sh under "Bitcoin maintainers/release engineers, update (commit) version in sources". Is that still part of the release process? If not, should release-process.md be amended to not mention verify.sh?

  2. MarcoFalke commented at 10:00 AM on February 25, 2016: member

    This is as trivial as bumping the two occurrences of 10.4 in https://github.com/bitcoin/bitcoin/commit/141c44ed6549acdc7f49fe039f017c020490ccf1 . If the script is still desired it would be better to create a python tool which automates such kind of trivial stuff, part of the release process.

  3. laanwj commented at 4:11 PM on March 1, 2016: member

    Can't we just add a VERSION parameter and tell the user to pass that in? Instead of relying it to be bumped in the source code with every release, which will be forgotten anyway.

  4. laanwj added the label Build system on Mar 1, 2016
  5. MarcoFalke commented at 8:10 PM on March 1, 2016: member

    It already does. You could use it like

    $ contrib/verifysfbinaries/verify.sh bitcoin-core-0.10.4-rc1
    $ contrib/verifysfbinaries/verify.sh bitcoin-core-0.11.2
    $ contrib/verifysfbinaries/verify.sh bitcoin-core-0.12.0
    

    Please note that it does not work for other rcs because their respective folder has been removed from https://bitcoin.org/bin/. (Interestingly, for 0.10.4 there are only rc binaries: https://bitcoin.org/bin/bitcoin-core-0.10.4/)

  6. laanwj commented at 12:54 PM on March 2, 2016: member

    OK, that's awesome, so we just remove the hardcoded value and update the documentation to call it in that way.

    Yes, rcs are removed from the server after the final release. There is very limited room on the server. 0.10.4 never left rc, and you're actually the first person to notice this.

  7. laanwj referenced this in commit 6d7ecb150b on Apr 15, 2016
  8. MarcoFalke referenced this in commit efa005fb92 on Apr 15, 2016
  9. laanwj referenced this in commit 3707e435be on Apr 15, 2016
  10. laanwj referenced this in commit 182bec4279 on Apr 25, 2016
  11. laanwj closed this on Apr 25, 2016

  12. codablock referenced this in commit 26c78a6b1b on Sep 19, 2017
  13. codablock referenced this in commit cec5a71f5c on Jan 9, 2018
  14. codablock referenced this in commit aed0b04133 on Jan 9, 2018
  15. lateminer referenced this in commit 50cd854f18 on Nov 9, 2018
  16. andvgal referenced this in commit 23695458cf on Jan 6, 2019
  17. MarcoFalke locked this on Sep 8, 2021

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: 2026-04-13 18:15 UTC

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