gitian: upgrade miniupnpc input to 1.9 #4033

pull laanwj wants to merge 2 commits into bitcoin:master from laanwj:2014_04_upgrade_miniupnpc changing 7 files +37 −41
  1. laanwj commented at 1:59 PM on April 9, 2014: member

    Bumps deps-linux, deps-win dependency versions as well.

    qt-win does not need to be bumped, as although it depends on deps-win, Qt doesn't use miniupnp. I verified this by rebuilding the dependency and checking the the output is the same. Not having to rebuild Qt is a good thing as it is huge.

  2. gitian: upgrade miniupnpc input to 1.9
    Bumps deps-linux, deps-win dependency versions as well.
    
    qt-win does not need to be bumped, as although it depends on deps-win,
    Qt doesn't use miniupnp. I verified this by rebuilding the dependency
    and checking the the output is the same. Not having to rebuild Qt is a
    good thing as it is huge.
    25d4911e86
  3. doc: update expected intermediate dependency hashes
    Also move them to release-process.md from gitian-building.md for more
    visibility.
    219b512d72
  4. ghost commented at 2:05 PM on April 9, 2014: none

    ACK gitian builds work for me with this patch

  5. BitcoinPullTester commented at 2:36 PM on April 9, 2014: none

    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/219b512d726a2cf73b2abf66eafb6fd236118d0d for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.

  6. gmaxwell commented at 3:58 PM on April 9, 2014: contributor

    ACK. (Untested)

  7. gavinandresen commented at 5:06 PM on April 9, 2014: contributor

    ACK (also untested). I did quickly review the miniupnpc 1.8-1.9 diff; besides scaring the pants off me with all the memcpy/strncpy, those changes look reasonable.

  8. laanwj commented at 5:26 PM on April 9, 2014: member

    I'm not very happy about miniupnpc either. My first hunch was to propose removing it completely. But I suppose we could lose quite some nodes if we dropped it.. (or at least, dropped upnp support, there are alternative libraries that could be used but I'm not sure they are safer)

  9. laanwj referenced this in commit 4c6cab2c5d on Apr 9, 2014
  10. laanwj merged this on Apr 9, 2014
  11. laanwj closed this on Apr 9, 2014

  12. 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-13 15:15 UTC

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