Fix and improve release-process.md #3028

pull Michagogo wants to merge 3 commits into bitcoin:master from Michagogo:fix-dep-links changing 1 files +12 −9
  1. Michagogo commented at 7:36 PM on September 24, 2013: contributor

    This commit adds a step, which is to git checkout the version to be built. This ensures that the gitian-descriptors for said version will be the correct ones. In addition, it updates the links for several dependencies, where the previously existing links were dead.

  2. in doc/release-process.md:None in 95013219d4 outdated
      41 | @@ -39,11 +42,11 @@ Release Process
      42 |  	wget 'http://miniupnp.free.fr/files/download.php?file=miniupnpc-1.6.tar.gz' -O miniupnpc-1.6.tar.gz
      43 |  	wget 'http://www.openssl.org/source/openssl-1.0.1c.tar.gz'
      44 |  	wget 'http://download.oracle.com/berkeley-db/db-4.8.30.NC.tar.gz'
      45 | -	wget 'http://zlib.net/zlib-1.2.6.tar.gz'
      46 | -	wget 'ftp://ftp.simplesystems.org/pub/libpng/png/src/libpng-1.5.9.tar.gz'
      47 | +	wget 'ftp://ftp.simplesystems.org/pub/libpng/png/src/history/zlib/zlib-1.2.6.tar.gz'
    


    Diapolo commented at 8:39 PM on September 24, 2013:

    AFAIK you shouldn't just update such links only here, they are used in the Gitian related files also (see folder gitian-descriptors).


    Michagogo commented at 9:55 PM on September 24, 2013:

    If I'm not mistaken, that's not the case -- as far as I can tell, any given gitian-build descriptor will get files from 2 potential sources. One is the remotes section, which specifies git repositories that should be downloaded, and the other is the files section, which lists files from the inputs directory that should be copied in. These wgets pull in the source that is used to build the deps. A quick look at the descriptor files, and I see no URLs there besides github.com/bitcoin/bitcoin.

    On Tuesday, September 24, 2013, Philip Kaufmann wrote:

    In doc/release-process.md:

    @@ -39,11 +42,11 @@ Release Process wget 'http://miniupnp.free.fr/files/download.php?file=miniupnpc-1.6.tar.gz' -O miniupnpc-1.6.tar.gz wget 'http://www.openssl.org/source/openssl-1.0.1c.tar.gz' wget 'http://download.oracle.com/berkeley-db/db-4.8.30.NC.tar.gz'

    • wget 'http://zlib.net/zlib-1.2.6.tar.gz'
    • wget 'ftp://ftp.simplesystems.org/pub/libpng/png/src/libpng-1.5.9.tar.gz'
    • wget 'ftp://ftp.simplesystems.org/pub/libpng/png/src/history/zlib/zlib-1.2.6.tar.gz'

    AFAIK you shouldn't just update such links only here, they are used in the Gitian related files also (see folder gitian-descriptors).

    — Reply to this email directly or view it on GitHubhttps://github.com/bitcoin/bitcoin/pull/3028/files#r6556043 .


    Diapolo commented at 5:45 AM on September 25, 2013:

    Seems I was wrong, sorry. I was just wondering, where the URIs that Gitian uses are coming from and had the impression this can't be the readme ;).

  3. sipa commented at 1:42 PM on September 29, 2013: member

    ACK changes, but needs rebase.

  4. Michagogo commented at 2:09 PM on September 29, 2013: contributor

    Rebased (I hope I did that right) and added one more small fix.

  5. in doc/release-process.md:None in 5e930fae47 outdated
      49 |  	wget 'http://fukuchi.org/works/qrencode/qrencode-3.2.0.tar.bz2'
      50 |  	wget 'http://downloads.sourceforge.net/project/boost/boost/1.54.0/boost_1_54_0.tar.bz2'
      51 |  	wget 'https://svn.boost.org/trac/boost/raw-attachment/ticket/7262/boost-mingw.patch' -O \ 
      52 |  	     boost-mingw-gas-cross-compile-2013-03-03.patch
      53 | -	wget 'http://releases.qt-project.org/qt4/source/qt-everywhere-opensource-src-4.8.3.tar.gz'
      54 | +	wget 'http://download.qt-project.org/archive/qt/4.8/4.8.3/qt-everywhere-opensource-src-4.8.3.tar.gz'
    


    Diapolo commented at 8:02 AM on October 1, 2013:

    Can you use https here and perhaps see if other downloads are also working via https?


    Michagogo commented at 6:16 PM on October 1, 2013:

    @Diapolo Okay, going down the list and adding https where it works -- is wget --no-check-certificate 'https://... any better than wget 'http://... ?

  6. Michagogo commented at 6:41 PM on October 1, 2013: contributor

    Switched from http to https for those downloads that support it, and rebased (a second time, onto Jeff's null wallet commit)

  7. Fix and improve release-process.md
    This commit adds a step, which is to git checkout the version to be
    built. This ensures that the gitian-descriptors for said version will be
    the correct ones. In addition, it updates the links for several
    dependencies, where the previously existing links were dead.
    7048443e8b
  8. Switch the order of win32 deps and win32 qt in release-process.md
    release-process.md has the deps being built after qt. However, the descriptors for qt call for the deps file.
    d18f5bf12c
  9. HTTPS-ify the downloads for the deps 61823c47db
  10. Michagogo commented at 9:03 PM on October 3, 2013: contributor

    Re-rebased.

  11. gavinandresen referenced this in commit 32b668655e on Oct 4, 2013
  12. gavinandresen merged this on Oct 4, 2013
  13. gavinandresen closed this on Oct 4, 2013

  14. Bushstar referenced this in commit 9041364b52 on Apr 8, 2020
  15. Bushstar referenced this in commit 781b165796 on Apr 8, 2020
  16. DrahtBot 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-19 15:15 UTC

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