depends: bump miniupnpc to 2.2.8 #30301

pull theuni wants to merge 1 commits into bitcoin:master from theuni:miniupnp-228-bump changing 4 files +6 −55
  1. theuni commented at 2:17 pm on June 18, 2024: member

    Drops two of our patches that have been merged upstream and adjusts the other to deal with recent changes.

    Follow-up from #30283. I can’t vouch for the upstream changes here.

  2. DrahtBot commented at 2:17 pm on June 18, 2024: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK Sjors
    Concept ACK edilmedeiros

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #30603 (Revert “depends: Fetch miniupnpc sources from an alternative website” by justinvforvendetta)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  3. DrahtBot added the label Build system on Jun 18, 2024
  4. fanquake added the label DrahtBot Guix build requested on Jun 18, 2024
  5. edilmedeiros commented at 4:56 pm on June 18, 2024: contributor

    Concept ACK.

    From the CI output, this has to wait for #30283 to be merged first, right?

  6. Sjors commented at 4:56 pm on June 18, 2024: member
    cc @laanwj
  7. DrahtBot added the label CI failed on Jun 18, 2024
  8. DrahtBot commented at 5:58 pm on June 18, 2024: contributor

    🚧 At least one of the CI tasks failed. Make sure to run all tests locally, according to the documentation.

    Possibly this is due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.

    Leave a comment here, if you need help tracking down a confusing failure.

    Debug: https://github.com/bitcoin/bitcoin/runs/26372834241

  9. theuni marked this as a draft on Jun 18, 2024
  10. theuni commented at 6:27 pm on June 18, 2024: member
    Marked as draft until #30283 is merged.
  11. DrahtBot commented at 0:52 am on June 19, 2024: contributor

    Guix builds (on x86_64) [untrusted test-only build, possibly unsafe, not for production use]

    File commit 9c5cdf07f30f816cd134e2cd2dca9c27ef7067a5(master) commit 9e8690801c64fb4037230dc672ce961df1e9129e(master and this pull)
    SHA256SUMS.part 8f176cd889e88c8c...
    *-aarch64-linux-gnu-debug.tar.gz c4603906c5e521c8...
    *-aarch64-linux-gnu.tar.gz 7f003e10e7ca2269...
    *-arm-linux-gnueabihf-debug.tar.gz e4a3366f4709760b...
    *-arm-linux-gnueabihf.tar.gz 5ce86cdc3b81597c...
    *-arm64-apple-darwin-unsigned.tar.gz e44ea6e8bee2b7f5...
    *-arm64-apple-darwin-unsigned.zip bd2feb2c203fd085...
    *-arm64-apple-darwin.tar.gz e6e67247faef3bd2...
    *-powerpc64-linux-gnu-debug.tar.gz 59c87ca578af3674...
    *-powerpc64-linux-gnu.tar.gz c42749de73d93831...
    *-riscv64-linux-gnu-debug.tar.gz 088a788947cf2c8e...
    *-riscv64-linux-gnu.tar.gz 7160ceb124bc9cf2...
    *-x86_64-apple-darwin-unsigned.tar.gz 78167426b180f468...
    *-x86_64-apple-darwin-unsigned.zip 779c80cc3c01c447...
    *-x86_64-apple-darwin.tar.gz 8b498a9181e91cdb...
    *-x86_64-linux-gnu-debug.tar.gz c44d3d010deda4b7...
    *-x86_64-linux-gnu.tar.gz 1700f4111e39a2b7...
    *.tar.gz 8ebf38cec0d3f4eb... d7f06d34f8704fdf...
    guix_build.log 554e151400868aa2... 527ed0648e64fcdc...
    guix_build.log.diff 36b8855325b9bbfa...
  12. DrahtBot removed the label DrahtBot Guix build requested on Jun 19, 2024
  13. fanquake commented at 8:40 am on June 20, 2024: member

    Marked as draft until #30283 is merged.

    Can rebase / undraft this now.

  14. depends: bump miniupnpc to 2.2.8 
    Drops two of our patches that have been merged upstream and adjusts the other
to deal with recent changes.
    545bb6c960
  15. theuni force-pushed on Jun 20, 2024
  16. theuni force-pushed on Jun 20, 2024
  17. theuni marked this as ready for review on Jun 20, 2024
  18. DrahtBot removed the label CI failed on Jun 21, 2024
  19. maflcko added the label DrahtBot Guix build requested on Jun 21, 2024
  20. TheCharlatan commented at 8:54 am on June 21, 2024: contributor

    Guix builds (aarch64):

     07f69ccde68d336bee6141d2cbcec8046332dedc908ba428ac08299daeb6a5d41  guix-build-545bb6c96080/output/aarch64-linux-gnu/SHA256SUMS.part
 1e843d7b6003da9f6259da41019f328af337858a45e4197ee9c5050984dc9e255  guix-build-545bb6c96080/output/aarch64-linux-gnu/bitcoin-545bb6c96080-aarch64-linux-gnu-debug.tar.gz
 21fdc9afbf83b2e8ce19d8a2c5cd080d87172ad02c4ce41036962f1bf675a415a  guix-build-545bb6c96080/output/aarch64-linux-gnu/bitcoin-545bb6c96080-aarch64-linux-gnu.tar.gz
 3400f2e70a689b6495af97def189cdf8b9d8dd740f5c45e8b851a4ff0469fc984  guix-build-545bb6c96080/output/arm-linux-gnueabihf/SHA256SUMS.part
 4e4d13ae0f2125606fdf456f7d62af25c9bec797f00c00e74c1236336d1174a67  guix-build-545bb6c96080/output/arm-linux-gnueabihf/bitcoin-545bb6c96080-arm-linux-gnueabihf-debug.tar.gz
 5c2370ff07929393f6ae735a12a3e2134e18b20acc7a534410be3da1461fda51a  guix-build-545bb6c96080/output/arm-linux-gnueabihf/bitcoin-545bb6c96080-arm-linux-gnueabihf.tar.gz
 682128f74a03ccd39ba027c7f608acbfbc327373d91a770eb58c883c1caf88c07  guix-build-545bb6c96080/output/arm64-apple-darwin/SHA256SUMS.part
 734a8894873b22acd65c2fdef4b80f8bbf9a930c651ed6d6307b8a43a2a0878c3  guix-build-545bb6c96080/output/arm64-apple-darwin/bitcoin-545bb6c96080-arm64-apple-darwin-unsigned.tar.gz
 804bc2a9d4ec1a2272687b0dee704ad0bc54944a3448b14e2a20af223a9050dc5  guix-build-545bb6c96080/output/arm64-apple-darwin/bitcoin-545bb6c96080-arm64-apple-darwin-unsigned.zip
 96db77ac9bf538d73038c8268af95fbb73d63c2045d174d4af54462fa61c42ec6  guix-build-545bb6c96080/output/arm64-apple-darwin/bitcoin-545bb6c96080-arm64-apple-darwin.tar.gz
10ada87843bc15e3e8ebba7ab3018bd2ea75d884f310acbd3dc9b6a01d5175be4c  guix-build-545bb6c96080/output/dist-archive/bitcoin-545bb6c96080.tar.gz
119affe451fbff49501cdc1fa1c076e110f8817654ce74edf36acbee401b2be840  guix-build-545bb6c96080/output/powerpc64-linux-gnu/SHA256SUMS.part
120dd5ff185fc27cf160a83b9e8bda93c9a975425d97312127161bec14011149b4  guix-build-545bb6c96080/output/powerpc64-linux-gnu/bitcoin-545bb6c96080-powerpc64-linux-gnu-debug.tar.gz
1368e380f13ca9a00a38b8a8046b3a1064f0c7d18801ece1bde26864c7f50de94c  guix-build-545bb6c96080/output/powerpc64-linux-gnu/bitcoin-545bb6c96080-powerpc64-linux-gnu.tar.gz
148480b2713395a42fa898ce9293c2b1d2cfd7196e18010be7ed30753e095c1c3e  guix-build-545bb6c96080/output/riscv64-linux-gnu/SHA256SUMS.part
1562446cea05d22dcad8f62475d3bb7220fa6c3423b294d8c9071f64fb11d107d1  guix-build-545bb6c96080/output/riscv64-linux-gnu/bitcoin-545bb6c96080-riscv64-linux-gnu-debug.tar.gz
169c23c77c50b1acacf90c4303e7b1fae4f6fe406bc00cea808248f2403def0219  guix-build-545bb6c96080/output/riscv64-linux-gnu/bitcoin-545bb6c96080-riscv64-linux-gnu.tar.gz
17685051ef7ba4793f90a66c5858266a309d937913171a3da0afc7335880d0dacc  guix-build-545bb6c96080/output/x86_64-apple-darwin/SHA256SUMS.part
189746a003bbcebb41d3a7cb365850f23b9364d5b3b25fda149a76437a5eb7e1e4  guix-build-545bb6c96080/output/x86_64-apple-darwin/bitcoin-545bb6c96080-x86_64-apple-darwin-unsigned.tar.gz
19ba77822a7b9f6082a5b5b41bfed51fe84e771b8e9869cb9a32421c263120796c  guix-build-545bb6c96080/output/x86_64-apple-darwin/bitcoin-545bb6c96080-x86_64-apple-darwin-unsigned.zip
2037fceeab62ead081699374400bf37442ff7f5f5a0712c67153dd436775cecab9  guix-build-545bb6c96080/output/x86_64-apple-darwin/bitcoin-545bb6c96080-x86_64-apple-darwin.tar.gz
216ee480721d582179a44487135160e0ce43cdfcbcddfd390d1fdb1cc6798628f5  guix-build-545bb6c96080/output/x86_64-linux-gnu/SHA256SUMS.part
22f30c17ffdaea58fb13180b53a676f7bc459f8ee473aa23af4e39b8fa1907e22e  guix-build-545bb6c96080/output/x86_64-linux-gnu/bitcoin-545bb6c96080-x86_64-linux-gnu-debug.tar.gz
230f0993cb2ad2eae0f2e843b47678969c290752f4bf082d2444e35dda87c4e86b  guix-build-545bb6c96080/output/x86_64-linux-gnu/bitcoin-545bb6c96080-x86_64-linux-gnu.tar.gz
24c45f789cb5400738afb0760e36bb02f75e048bb4e680d073661d56f5597194d8  guix-build-545bb6c96080/output/x86_64-w64-mingw32/SHA256SUMS.part
25765b5e54a254fbf22e8e54753cfa46029940596daec291394d055232382005b4  guix-build-545bb6c96080/output/x86_64-w64-mingw32/bitcoin-545bb6c96080-win64-debug.zip
26551285d885d75e4d263bc12517a941fca4601f7fd9dd034adc7eea6c0ae33268  guix-build-545bb6c96080/output/x86_64-w64-mingw32/bitcoin-545bb6c96080-win64-setup-unsigned.exe
27d59df500857d2ee8a1d08a9780162192f19dff2d4585dc827575c6a9fba274ae  guix-build-545bb6c96080/output/x86_64-w64-mingw32/bitcoin-545bb6c96080-win64-unsigned.tar.gz
2849ad4a2568e699ac4a00e472926e77fc1e23cf66c4ce511ce70d4149a1204ac4  guix-build-545bb6c96080/output/x86_64-w64-mingw32/bitcoin-545bb6c96080-win64.zip
  21. Sjors commented at 12:58 pm on June 21, 2024: member

    tACK 545bb6c96080

    I looked at the diff between the packaged 2.2.7 and 2.2.8. There’s a MINIUPNPC_API_VERSION bump from 17 to 18, which was addressed in #30283. I also see our dropped patches reflected in the diff. Didn’t study the rest of the changes.

    Tested on macOS 14.5 with -upnp=1 -natpmp=0.

    When I turn UPnP support off in the router:

    0[mapport] No valid UPnP IGDs found

    When I turn it on:

    02024-06-21T12:55:33Z [mapport] UPnP: ExternalIPAddress = x.x.x.x
12024-06-21T12:55:33Z [mapport] AddLocal(x.x.x.x:x,3)
22024-06-21T12:55:33Z [mapport] UPnP Port Mapping successful.

    My OPNSense router identifies this as Bitcoin Core v27.99.0-g545bb6c960800760512634a7227337fa779ea944. Same behavior as with the v27.0 release binary.

    So I’m not sure what depends/patches/miniupnpc/dont_leak_info.patch is supposed to achieve.

    Guix hashes:

     07f69ccde68d336bee6141d2cbcec8046332dedc908ba428ac08299daeb6a5d41  guix-build-545bb6c96080/output/aarch64-linux-gnu/SHA256SUMS.part
 1e843d7b6003da9f6259da41019f328af337858a45e4197ee9c5050984dc9e255  guix-build-545bb6c96080/output/aarch64-linux-gnu/bitcoin-545bb6c96080-aarch64-linux-gnu-debug.tar.gz
 21fdc9afbf83b2e8ce19d8a2c5cd080d87172ad02c4ce41036962f1bf675a415a  guix-build-545bb6c96080/output/aarch64-linux-gnu/bitcoin-545bb6c96080-aarch64-linux-gnu.tar.gz
 3400f2e70a689b6495af97def189cdf8b9d8dd740f5c45e8b851a4ff0469fc984  guix-build-545bb6c96080/output/arm-linux-gnueabihf/SHA256SUMS.part
 4e4d13ae0f2125606fdf456f7d62af25c9bec797f00c00e74c1236336d1174a67  guix-build-545bb6c96080/output/arm-linux-gnueabihf/bitcoin-545bb6c96080-arm-linux-gnueabihf-debug.tar.gz
 5c2370ff07929393f6ae735a12a3e2134e18b20acc7a534410be3da1461fda51a  guix-build-545bb6c96080/output/arm-linux-gnueabihf/bitcoin-545bb6c96080-arm-linux-gnueabihf.tar.gz
 682128f74a03ccd39ba027c7f608acbfbc327373d91a770eb58c883c1caf88c07  guix-build-545bb6c96080/output/arm64-apple-darwin/SHA256SUMS.part
 734a8894873b22acd65c2fdef4b80f8bbf9a930c651ed6d6307b8a43a2a0878c3  guix-build-545bb6c96080/output/arm64-apple-darwin/bitcoin-545bb6c96080-arm64-apple-darwin-unsigned.tar.gz
 804bc2a9d4ec1a2272687b0dee704ad0bc54944a3448b14e2a20af223a9050dc5  guix-build-545bb6c96080/output/arm64-apple-darwin/bitcoin-545bb6c96080-arm64-apple-darwin-unsigned.zip
 96db77ac9bf538d73038c8268af95fbb73d63c2045d174d4af54462fa61c42ec6  guix-build-545bb6c96080/output/arm64-apple-darwin/bitcoin-545bb6c96080-arm64-apple-darwin.tar.gz
10ada87843bc15e3e8ebba7ab3018bd2ea75d884f310acbd3dc9b6a01d5175be4c  guix-build-545bb6c96080/output/dist-archive/bitcoin-545bb6c96080.tar.gz
119affe451fbff49501cdc1fa1c076e110f8817654ce74edf36acbee401b2be840  guix-build-545bb6c96080/output/powerpc64-linux-gnu/SHA256SUMS.part
120dd5ff185fc27cf160a83b9e8bda93c9a975425d97312127161bec14011149b4  guix-build-545bb6c96080/output/powerpc64-linux-gnu/bitcoin-545bb6c96080-powerpc64-linux-gnu-debug.tar.gz
1368e380f13ca9a00a38b8a8046b3a1064f0c7d18801ece1bde26864c7f50de94c  guix-build-545bb6c96080/output/powerpc64-linux-gnu/bitcoin-545bb6c96080-powerpc64-linux-gnu.tar.gz
148480b2713395a42fa898ce9293c2b1d2cfd7196e18010be7ed30753e095c1c3e  guix-build-545bb6c96080/output/riscv64-linux-gnu/SHA256SUMS.part
1562446cea05d22dcad8f62475d3bb7220fa6c3423b294d8c9071f64fb11d107d1  guix-build-545bb6c96080/output/riscv64-linux-gnu/bitcoin-545bb6c96080-riscv64-linux-gnu-debug.tar.gz
169c23c77c50b1acacf90c4303e7b1fae4f6fe406bc00cea808248f2403def0219  guix-build-545bb6c96080/output/riscv64-linux-gnu/bitcoin-545bb6c96080-riscv64-linux-gnu.tar.gz
17685051ef7ba4793f90a66c5858266a309d937913171a3da0afc7335880d0dacc  guix-build-545bb6c96080/output/x86_64-apple-darwin/SHA256SUMS.part
189746a003bbcebb41d3a7cb365850f23b9364d5b3b25fda149a76437a5eb7e1e4  guix-build-545bb6c96080/output/x86_64-apple-darwin/bitcoin-545bb6c96080-x86_64-apple-darwin-unsigned.tar.gz
19ba77822a7b9f6082a5b5b41bfed51fe84e771b8e9869cb9a32421c263120796c  guix-build-545bb6c96080/output/x86_64-apple-darwin/bitcoin-545bb6c96080-x86_64-apple-darwin-unsigned.zip
2037fceeab62ead081699374400bf37442ff7f5f5a0712c67153dd436775cecab9  guix-build-545bb6c96080/output/x86_64-apple-darwin/bitcoin-545bb6c96080-x86_64-apple-darwin.tar.gz
216ee480721d582179a44487135160e0ce43cdfcbcddfd390d1fdb1cc6798628f5  guix-build-545bb6c96080/output/x86_64-linux-gnu/SHA256SUMS.part
22f30c17ffdaea58fb13180b53a676f7bc459f8ee473aa23af4e39b8fa1907e22e  guix-build-545bb6c96080/output/x86_64-linux-gnu/bitcoin-545bb6c96080-x86_64-linux-gnu-debug.tar.gz
230f0993cb2ad2eae0f2e843b47678969c290752f4bf082d2444e35dda87c4e86b  guix-build-545bb6c96080/output/x86_64-linux-gnu/bitcoin-545bb6c96080-x86_64-linux-gnu.tar.gz
24c45f789cb5400738afb0760e36bb02f75e048bb4e680d073661d56f5597194d8  guix-build-545bb6c96080/output/x86_64-w64-mingw32/SHA256SUMS.part
25765b5e54a254fbf22e8e54753cfa46029940596daec291394d055232382005b4  guix-build-545bb6c96080/output/x86_64-w64-mingw32/bitcoin-545bb6c96080-win64-debug.zip
26551285d885d75e4d263bc12517a941fca4601f7fd9dd034adc7eea6c0ae33268  guix-build-545bb6c96080/output/x86_64-w64-mingw32/bitcoin-545bb6c96080-win64-setup-unsigned.exe
27d59df500857d2ee8a1d08a9780162192f19dff2d4585dc827575c6a9fba274ae  guix-build-545bb6c96080/output/x86_64-w64-mingw32/bitcoin-545bb6c96080-win64-unsigned.tar.gz
2849ad4a2568e699ac4a00e472926e77fc1e23cf66c4ce511ce70d4149a1204ac4  guix-build-545bb6c96080/output/x86_64-w64-mingw32/bitcoin-545bb6c96080-win64.zip
  22. DrahtBot commented at 2:56 pm on June 21, 2024: contributor

    Guix builds (on x86_64) [untrusted test-only build, possibly unsafe, not for production use]

    File commit a961ad1bebc54912b88d072abf22ab7d3cf46bf1(master) commit 870a7018f3aaf6021be3dee8d1d727f1d2f9fb93(master and this pull)
    SHA256SUMS.part 04a15c0a10429bbe... 04535b45813bffd1...
    *-aarch64-linux-gnu-debug.tar.gz e13d44f63a2b7ac2... 03ab99e23f85a3a0...
    *-aarch64-linux-gnu.tar.gz 37fdb7cc5b771ca5... 60ea95637bb56703...
    *-arm-linux-gnueabihf-debug.tar.gz 33f2880d5cb2afcf... 73a811b7e7975369...
    *-arm-linux-gnueabihf.tar.gz 0b61520b80662b09... b44898bf31160864...
    *-arm64-apple-darwin-unsigned.tar.gz 1527d594a9312b99... 96941bcd430f3c14...
    *-arm64-apple-darwin-unsigned.zip e2ed7ee21cc9e83c... f98e5dec03b00896...
    *-arm64-apple-darwin.tar.gz 0ae67d0a47b65c5f... cabda1ae9f5d0766...
    *-powerpc64-linux-gnu-debug.tar.gz ff39468cd1f74b57... 6acd0623343efd3e...
    *-powerpc64-linux-gnu.tar.gz 94d99bf1aa2ee405... c37335ac8615434a...
    *-riscv64-linux-gnu-debug.tar.gz 2ba20b9f3c47d3f8... 150836979103e947...
    *-riscv64-linux-gnu.tar.gz 966031a76fd20d13... 4b299ecaad70eea2...
    *-x86_64-apple-darwin-unsigned.tar.gz 8f9e6f84d4f81f2a... 8c73350d7c01b299...
    *-x86_64-apple-darwin-unsigned.zip 15ebb77a6f77345e... a8c9983687f245f0...
    *-x86_64-apple-darwin.tar.gz 82e66276aef7d70a... fcf394d8ad9eed97...
    *-x86_64-linux-gnu-debug.tar.gz 4ca33fbe0ccc5faf... 16a166f711dd36de...
    *-x86_64-linux-gnu.tar.gz 4fbcde5d6e5b8a61... 5357c58f683d473f...
    *.tar.gz 59b7e589b6f90976... 0b6374b10e70c01f...
    guix_build.log c2550293c728bf3c... d67d70c6f8150a31...
    guix_build.log.diff df63483c70a9896c...
  23. DrahtBot removed the label DrahtBot Guix build requested on Jun 21, 2024
  24. DrahtBot added the label Needs rebase on Aug 12, 2024
  25. DrahtBot commented at 12:23 pm on August 12, 2024: contributor

    🐙 This pull request conflicts with the target branch and needs rebase.

  26. edilmedeiros commented at 5:03 pm on October 23, 2024: contributor
    Just curious if this is still relevant after #30043? Or probably #31130?
  27. fanquake commented at 8:20 am on October 24, 2024: member

    Or probably #31130?

    If #31130 is merged, then yes, this is no-longer relevant. Drafted for now, given this also needs rebase.

  28. fanquake marked this as a draft on Oct 24, 2024
  29. fanquake commented at 9:28 am on October 25, 2024: member
    I think #31130 is going to go in for v29.0. If that doesn’t happen, we can reopen here.
  30. fanquake closed this on Oct 25, 2024
  31. laanwj commented at 9:36 am on October 25, 2024: member

    I think #31130 is going to go in for v29.0. If that doesn’t happen, we can reopen here.

    yes… i dont think i’ve seen any PR that (uncontroversially) popular in my time here 😄

  32. theuni commented at 3:59 pm on October 25, 2024: member
    Woohoo! I like that much better than a bump :)


theuni DrahtBot edilmedeiros Sjors fanquake TheCharlatan laanwj

Labels
Build system Needs rebase

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: 2024-11-21 12:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me