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. MarcoFalke cross-referenced this on Mar 14, 2016 from issue [WIP] Backports for 0.10.5 (updated to 93ca5a3) by luke-jr
  8. laanwj referenced this in commit 6d7ecb150b on Apr 15, 2016
  9. laanwj cross-referenced this on Apr 15, 2016 from issue Update release process by laanwj
  10. MarcoFalke referenced this in commit efa005fb92 on Apr 15, 2016
  11. laanwj referenced this in commit 3707e435be on Apr 15, 2016
  12. laanwj referenced this in commit 182bec4279 on Apr 25, 2016
  13. laanwj closed this on Apr 25, 2016

  14. codablock referenced this in commit 26c78a6b1b on Sep 19, 2017
  15. codablock referenced this in commit cec5a71f5c on Jan 9, 2018
  16. codablock referenced this in commit aed0b04133 on Jan 9, 2018
  17. lateminer referenced this in commit 50cd854f18 on Nov 9, 2018
  18. andvgal referenced this in commit 23695458cf on Jan 6, 2019
  19. bitcoin 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-05-20 06:55 UTC