depends: Fix zeromq build on OpenBSD #30565

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:240731-openbsd-zmq changing 2 files +26 −0
  1. hebasto commented at 10:45 pm on July 31, 2024: member

    On the master branch @ 66e82dc90c598c9c42ff980693ef5367a845e1d0, the zeromq package fails to build on OpenBSD 7.5:

    0[ 19%] Building CXX object CMakeFiles/objects.dir/src/io_thread.cpp.o
    1/home/hebasto/bitcoin/depends/work/build/amd64-unknown-openbsd7.5/zeromq/4.3.5-df5b1b9f936/src/io_thread.cpp:14:22: error: static_cast from 'std::nullptr_t' to 'poller_t::handle_t' (aka 'int') is not allowed
    2    _mailbox_handle (static_cast<poller_t::handle_t> (NULL))
    3                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    41 error generated.
    

    This regression was overlooked by me in #29723.

    This PR fixes the issue by backporting an upstream commit from https://github.com/zeromq/libzmq/pull/4659.

  2. depends: Fix `zeromq` build on OpenBSD 89b1d5c818
  3. hebasto added the label Build system on Jul 31, 2024
  4. DrahtBot commented at 10:45 pm on July 31, 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 theStack

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

  5. hebasto commented at 10:45 pm on July 31, 2024: member
    Friendly ping @theStack :)
  6. hebasto added this to the milestone 28.0 on Aug 1, 2024
  7. theStack approved
  8. theStack commented at 1:51 am on August 1, 2024: contributor

    tACK 89b1d5c818a33440c0c497aa78721fb9b0af35a5

    Verified on OpenBSD 7.5 that building the zeromq depends build via

    0$ gmake -C depends NO_BOOST=1 NO_LIBEVENT=1 NO_QT=1 NO_SQLITE=1 NO_NATPMP=1 NO_UPNP=1 NO_USDT=1 NO_BDB=1
    

    fails on master (same error message as in the PR description) and succeeds on the PR branch. Checked also that the introduced patch matches https://github.com/zeromq/libzmq/pull/4659/commits/f9a798c9fb26124206ae2a3e72225f6d9dd8c646.

  9. fanquake merged this on Aug 1, 2024
  10. fanquake closed this on Aug 1, 2024

  11. hebasto deleted the branch on Aug 1, 2024


hebasto DrahtBot theStack

Labels
Build system

Milestone
28.0


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 21:12 UTC

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