build: Do not use QtDBus for macOS builds #18045

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:20200131-macos-qdbus changing 1 files +4 −0
  1. hebasto commented at 5:17 PM on January 31, 2020: member

    This is an alternative to #18042:

    When cross compiling for macOS with depends, we do not compile with dbus. ... This fixes the error by always avoiding DBUS notification on macOS.

    Another (eventually better) way to fix this would be to avoid detection of host qt pkg-config packages when cross compiling.

  2. DrahtBot added the label Build system on Jan 31, 2020
  3. hebasto force-pushed on Jan 31, 2020
  4. jonasschnelli cross-referenced this on Jan 31, 2020 from issue Don't check for DBUS notification on macOS by jonasschnelli
  5. DrahtBot commented at 5:55 PM on January 31, 2020: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    No conflicts as of last run.

  6. jonasschnelli commented at 5:56 PM on January 31, 2020: contributor

    Looks like it doesn't fix the issue: https://bitcoinbuilds.org/logs/251e6256-d6a5-455e-aed8-e39338fd1882.log (somehow still compiles with USE_DBUS)

  7. build: Do not use QtDBus for macOS builds f1b20ea0e9
  8. hebasto force-pushed on Jan 31, 2020
  9. hebasto commented at 6:05 PM on January 31, 2020: member

    @jonasschnelli

    Looks like it doesn't fix the issue: https://bitcoinbuilds.org/logs/251e6256-d6a5-455e-aed8-e39338fd1882.log (somehow still compiles with USE_DBUS)

    Fixed.

  10. jonasschnelli approved
  11. jonasschnelli commented at 7:48 PM on January 31, 2020: contributor

    Tested ACK f1b20ea0e9918e1566112a94d8c5360246b7b235 - there is no need to use dbus on macOS

  12. fanquake commented at 1:28 AM on February 1, 2020: member

    Isn't this just masking the actual issue, which is that configure is finding QtDBus outside of what we've built in depends?

    Why are we special casing macOS? Shouldn't this same issue be affecting essentially all targets where we explicitly disable dbus when building qt?

  13. jonasschnelli commented at 7:45 AM on February 1, 2020: contributor

    Kinda agree with @fanquake that fixing the pkg-config path for depends builds should be done.

    Though this quick fix does also make sense on its own.

  14. hebasto cross-referenced this on Feb 1, 2020 from issue build: Fix behavior when ALLOW_HOST_PACKAGES unset by hebasto
  15. hebasto commented at 12:13 PM on February 1, 2020: member

    Closed in favor of #18051.

  16. hebasto closed this on Feb 1, 2020

  17. hebasto cross-referenced this on Feb 3, 2020 from issue build: macOS toolchain update by fanquake
  18. laanwj referenced this in commit 646f0ada02 on Feb 10, 2020
  19. sidhujag referenced this in commit 47fe769fc5 on Feb 18, 2020
  20. hebasto deleted the branch on Jun 13, 2020
  21. sidhujag referenced this in commit e9ea7cd6e6 on Nov 10, 2020
  22. bitcoin locked this on Feb 15, 2022

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-19 06:54 UTC